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

Skip to content

[Mailer] max_per_second option configurable via DSN #46315

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 20, 2022

Conversation

gassan
Copy link
Contributor

@gassan gassan commented May 11, 2022

Q A
Branch? 6.2
Bug fix? no
New feature? yes
Deprecations? no
Tickets -
License MIT
Doc PR symfony/symfony-docs#16800

This option is available since 5.1, but it was impossible to configure it by dsn.

@carsonbot carsonbot added this to the 5.4 milestone May 11, 2022
@gassan gassan changed the title made max_per_second option configurable by dsn. [Mailer] made max_per_second option configurable by dsn. May 11, 2022
@nicolas-grekas nicolas-grekas modified the milestones: 5.4, 6.2 May 11, 2022
@nicolas-grekas
Copy link
Member

This should be considered as anew feature, so it should target 6.2 (branch 6.1 for now)

@gassan gassan changed the base branch from 5.4 to 6.1 May 11, 2022 12:18
@lyrixx lyrixx removed their request for review May 11, 2022 16:15
@fabpot
Copy link
Member

fabpot commented May 13, 2022

Is there anything else interesting that could be added to the DSN?

@OskarStark OskarStark changed the title [Mailer] made max_per_second option configurable by dsn. [Mailer] max_per_second option configurable via DSN May 20, 2022
@fabpot
Copy link
Member

fabpot commented Jul 20, 2022

Thank you @gassan.

@fabpot fabpot merged commit 397abb6 into symfony:6.2 Jul 20, 2022
@fabpot fabpot mentioned this pull request Oct 24, 2022
@gassan gassan deleted the max-per-second branch January 5, 2023 09:27
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.

5 participants