Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[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
mailer.rst Show resolved Hide resolved
mailer.rst Show resolved Hide resolved
mailer.rst Outdated Show resolved Hide resolved
@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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.