When both of the above modules are installed the user_status mails are not displayed correctly because the headers is not correct.

The header in user_status is set to:

$headers = "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: <$from>\nErrors-to: $from\n";

The last line break will create an emtpy line in the header which will be parsed by the SMTP module and output a single ":" which will break the header and show parts of the header in the body of the email message.

After checking the user.module from drupal core i replaced it with the following line which resolves the issue:

$headers = "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: $from\nErrors-to: $from";

Patch included.

Carlo

CommentFileSizeAuthor
smtp_user_status.patch1.36 KBclandmeter

Comments

dww’s picture

Status: Active » Fixed

good catch. reviewed, tested, and committed to DRUPAL-4-6 and DRUPAL-4-7.

thanks!
-derek

p.s. in the future, if you submit issues with patches attached, please set the status to "patch (code needs review)". thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)