Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send SMTPUTF8 if the message needs it and the server supports it #20

Closed
wants to merge 3 commits into from

Conversation

arnt
Copy link

@arnt arnt commented Sep 2, 2024

No description provided.

Before this commit, Envelope would throw InvalidArgumentException when a
unicode sender address was used. Now, that error is thrown slightly later,
is thrown for recipient addresses as well, but is not thrown if the
next-hop server supports SMTPUTF8.

As a side effect, transports that use JSON APIs to ESPs can also use
unicode addresses if the ESP supports that (many do, many don't).
@GromNaN
Copy link
Member

GromNaN commented Sep 21, 2024

Thank you for contributing. Your pull request must be opened on the main symfony/symfony repository. This repository is a subtree mirror of the component.

@arnt
Copy link
Author

arnt commented Sep 23, 2024

Thanks for the hint. I made a PR there now, including a CHANGELOG.md entry. I'll close this and my mime PR, and delete the source repos.

@arnt arnt closed this Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants