Skip to content

[Mailer] [Smtp] Add DSN param to enforce TLS/STARTTLS #20644

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.

Sign up for GitHub

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

Jump to bottom
Closed
nicolas-grekas opened this issue Feb 11, 2025 · 0 comments
Closed

[Mailer] [Smtp] Add DSN param to enforce TLS/STARTTLS #20644

nicolas-grekas opened this issue Feb 11, 2025 · 0 comments
Labels
hasPR A Pull Request has already been submitted for this issue. Mailer
Milestone
7.3

Comments

@nicolas-grekas
Copy link
Member

Q A
Feature PR symfony/symfony#59479
PR author(s) @ssddanbrown
Merged in 7.3

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 7.3 branch in the symfony-docs repository.

Thank you! 😃
@nicolas-grekas nicolas-grekas added this to the 7.3 milestone Feb 11, 2025
ssddanbrown added a commit to ssddanbrown/symfony-docs that referenced this issue Mar 1, 2025
@ssddanbrown
[Mailer] Document require_tls option
e1048bb 
For symfony#20644
@ssddanbrown ssddanbrown mentioned this issue Mar 1, 2025
Merged
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Mar 1, 2025
javiereguiluz added a commit that referenced this issue May 8, 2025
@javiereguiluz
minor #20701 [Mailer] Document require_tls option (ssddanbrown)
7d360e6 
This PR was squashed before being merged into the 7.3 branch.

Discussion
----------

[Mailer] Document `require_tls` option

For #20644. This PR documents the added SMTP `require_tls` option, as introduced in symfony/symfony#59479.

I have referenced that this could throw a `TransportException` (where TLS is not achieved) but not sure if I should instead reference `TransportExceptionInterface`.

Commits
-------

55217dc [Mailer] Document `require_tls` option
@xabbuh xabbuh closed this as completed May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue. Mailer
Projects
None yet
Milestone
7.3
Development

No branches or pull requests
3 participants
@nicolas-grekas @xabbuh @carsonbot