Closed (fixed)
Project:
User status change notifications
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Jan 2007 at 12:08 UTC
Updated:
20 Jan 2007 at 08:46 UTC
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
| Comment | File | Size | Author |
|---|---|---|---|
| smtp_user_status.patch | 1.36 KB | clandmeter |
Comments
Comment #1
dwwgood 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.
Comment #2
(not verified) commented