-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Explicitly decide whether to use TLS in mailer's configuration #5024
Explicitly decide whether to use TLS in mailer's configuration #5024
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5024 +/- ##
==========================================
+ Coverage 37.45% 37.45% +<.01%
==========================================
Files 312 312
Lines 46510 46510
==========================================
+ Hits 17419 17422 +3
+ Misses 26599 26597 -2
+ Partials 2492 2491 -1
Continue to review full report at Codecov.
|
It should probably use default depending on port set to not break compatibility |
…citly_use_tls_for_mailer
4fd1d20
to
94a93ab
Compare
Sorry for not updating this on time. I have been a little busy this past 2 weeks. I would be updating my other PRs before the start of next week |
@adelowo no worries. We appreciate the time you are able to give. |
Fixes #4820