-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[Mailer] Make RoundRobinTransport
retryPeriod
configurable
#50981
Comments
Thank you for this suggestion. |
Yes please |
Would be very useful |
PR welcome |
With something configurable like |
@sdespont yes |
Is this something that must be in a DSN ? The retry_period won't change between instances/servers right ? Should this be configured elsewhere / userland / container ? |
Yes it should be in DSN IMHO, you should be able to change that in the config without a new commit. |
Description
I started using the
roundrobin(...)
config option of the Mailer, but I'm unable to configure theretryPeriod
option, which would grant retries over failed connections:symfony/src/Symfony/Component/Mailer/Transport/RoundRobinTransport.php
Line 38 in 80f1096
This is an issue while using the Messenger (which I'm doing, even if not directly) because for 60 seconds my consumer is susceptible to any kind of transport failure.
I tried overriding that, but it required decorating the transport factory and using reflection, so... adding the possibility of piloting that option would be great!
Example
No response
The text was updated successfully, but these errors were encountered: