Posted by pillarsdotnet on June 25, 2011 at 4:49pm
5 followers
| Project: | SMTP Authentication Support |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
While troubleshooting a bug report, I discovered that SmtpMailSystem::mail() is incorrectly parsing the Content-Type header.
Comments
#1
This patch allows SMTP, HTML Mail, and Mail System to work together.
#2
Created a documentation page illustrating the use-case that makes the above patch necessary.
#3
Also, it would be lovely if the SMTP developers would contribute to the documentation here.
#4
Reviewed and tested, but the middle part of the patch will conflict with #1330714: Character encoding conflict between Drupal and phpmailer., which when applied will solve this issue more elegant.
Why this patch works:
Content-Type can be setup like "text/plain; foo/bar", but it can also be "text/plain;foo/bar". The space is not a must and such should not be accounted for.
The patch also improves the error message by saying, which header could not be analyzed. This will help in debugging.
Please apply.
#5
Could someone re-roll after the fix on #1330714: Character encoding conflict between Drupal and phpmailer. ?
#6
Re-rolled patch by omitting middle part, which was solved in #1330714: Character encoding conflict between Drupal and phpmailer..
=> Back to RTBC
#7
Thank you for patches and tests. Committed.
#8
Automatically closed -- issue fixed for 2 weeks with no activity.