Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Mailer] ping_threshold for ses+smtp #15557

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

Merged
merged 1 commit into from
Jul 29, 2021
Merged

Conversation

Tyraelqp
Copy link
Contributor

fabpot added a commit to symfony/symfony that referenced this pull request Jul 26, 2021
…Factory (Tyraelqp)

This PR was squashed before being merged into the 5.4 branch.

Discussion
----------

[Mailer] Add support of ping_threshold to SesTransportFactory

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | Yes
| Deprecations? | no
| Tickets       | Fix #39044
| License       | MIT
| Doc PR        | symfony/symfony-docs#15557

Added support of ping_threshold option to `SesTransportFactory` for `ses+smtp` and `ses+smtps` schemes. Needed because SES closes SMTP connection after 10 seconds of inactivity and `TransportException` will be thrown on next send: `Expected response code "250" but got code "451", with message "451 4.4.2 Timeout waiting for data from client.".`

Commits
-------

2e50135 [Mailer] Add support of ping_threshold to SesTransportFactory
@javiereguiluz javiereguiluz merged commit ed0771b into symfony:5.4 Jul 29, 2021
@javiereguiluz
Copy link
Member

Sergey, thanks a lot for contributing this feature and for contributing the docs too. It was a great first contribution to Symfony Docs! Congrats on that 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants