User Notifications Not Working

Resolved bbell2000
(@bbell2000)

6 years, 4 months ago

Pie Register 3.4.2 is installed and fully configured on WP 5.3.2.

After registering a test user requiring admin approval, I received the admin notification. However, the user did *not* receive any notification for neither registration nor verification. All notifications are enabled and configured.

The same from address is used for both admin and user notifications so I know that’s not the issue.

I’ve confirmed that the user’s email address is correct and I’ve verified that my WP is able to send email to that user.

My WP is configured to send email via the WP Offload SES Lite plug version 1.3. The SES plug-in *is* attempting to send the email, but it’s failing. The only way I know of that this can happen is if Pie Register is using the wrong (unverified) from address or wrong domain (e.g., http://www.example.com instead of example.com).

Given that the admin notifications are working, there has to be something different going on behind the scenes for user notifications.

Suggestions for troubleshooting? I have debug logging enabled and there’s not much help in the logs.

Viewing 5 replies - 1 through 5 (of 5 total)

Thread Starter bbell2000
(@bbell2000)

6 years, 4 months ago

I now see I have two lines in my logs that coincide with the send error:

[09-Feb-2020 15:57:13 UTC] PHP Notice: Undefined variable: slug in [snip]/wp-content/plugins/pie-register/classes/registration_form.php on line 663
[09-Feb-2020 15:57:13 UTC] PHP Notice: Trying to get property of non-object in [snip]/wp-content/plugins/pie-register/classes/registration_form.php on line 738
Thread Starter bbell2000
(@bbell2000)

6 years, 4 months ago
Ok… I think I’ve found the problem. I’m pretty sure there’s a bug in pie-register.php. Actually, several instances of the same bug.

I’m pretty sure sending via SES is a bit stricter than normal SMTP. php-register.php is generating raw emails with two “Return-Path” headers. Sometimes, the header is set to a valid email address. Otherwise, it’s set to $from_name. That’s illegal as per the SMTP spec.

Example from php-register.php line 3311:
```
if($reply_email){
  $headers .= "Reply-To: {$reply_email}\r\n";
  $headers .= "Return-Path: {$from_name}\r\n";
 }else{
  $headers .= "Reply-To: {$from_email}\r\n";
  $headers .= "Return-Path: {$from_email}\r\n";
}
```
Thread Starter bbell2000
(@bbell2000)

6 years, 4 months ago

Bug verified. I had to change the invalid Return-Path header values in classes/edit_form.php, forgot_password.php, login_form.php and pie-register.php.

All user-facing emails are now working with the SES plugin.

Plugin Contributor genetech
(@genetech)

6 years, 4 months ago

Hi,

Thanks for reporting this bug. We will update it in our next version.

Best

Plugin Contributor genetech
(@genetech)

6 years, 4 months ago

This fix is released. Please update your plugin to latest version (3.4.4)

Viewing 5 replies - 1 through 5 (of 5 total)

The topic ‘User Notifications Not Working’ is closed to new replies.

Tags