Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Mailer] maintain sender/recipient name in SMTP envelopes#33387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
xabbuh commentedAug 30, 2019
| Q | A |
|---|---|
| Branch? | 4.4 |
| Bug fix? | yes |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #33383 |
| License | MIT |
| Doc PR |
xabbuh commentedAug 30, 2019
The build failure is unrelated to the changes. |
18339a0 toc1d2451Comparexabbuh commentedAug 30, 2019
I found some more places. Ready to be reviewed now. |
fbourigault commentedAug 30, 2019 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
As Moreover, we are lacking some escaping to flawlessly use addresses names in email headers. AFAIK, commas and double quotes have to be escaped. I use the following code in my project which ATM use the official postmark client: $name =preg_replace('/"/u','\"',$name);if (preg_match('/,/u',$name)) {$name =sprintf('"%s"',$name); } I quickly looked at RFC, but address grammar can be found inhttps://tools.ietf.org/html/rfc2822#section-3.4. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
fabpot commentedSep 3, 2019
I think this one needs to be rebased on 4.3 as this is a bug fix, right?@xabbuh Can you rebase? |
fabpot commentedSep 4, 2019
Thank you@xabbuh. |
…(xabbuh)This PR was merged into the 4.3 branch.Discussion----------[Mailer] maintain sender/recipient name in SMTP envelopes| Q | A| ------------- | ---| Branch? | 4.4| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#33383| License | MIT| Doc PR |Commits-------46ed0e8 maintain sender/recipient name in SMTP envelopes