Skip to content

Comments

Document symfony mailer#10023

Merged
miaulalala merged 1 commit intomasterfrom
enh/document-symfony
Apr 13, 2023
Merged

Document symfony mailer#10023
miaulalala merged 1 commit intomasterfrom
enh/document-symfony

Conversation

@miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Apr 6, 2023

Fixes #9600

Before After
screencapture-127-0-0-1-8000-configuration-server-email-configuration-html-2023-04-06-21_42_23 screencapture-127-0-0-1-8000-configuration-server-email-configuration-html-2023-04-12-12_56_43

@miaulalala miaulalala self-assigned this Apr 6, 2023
@miaulalala miaulalala force-pushed the enh/document-symfony branch 2 times, most recently from e6e0f31 to d1537ab Compare April 6, 2023 19:34
@miaulalala miaulalala changed the title Enh/document symfony Document symfony mailer Apr 6, 2023
@miaulalala
Copy link
Contributor Author

/backport to stable26

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The section regarding manual TLS configuration is not relevant anymore and can be removed. AFAIK, STARTTLS is used automatically and can't be enforced (as stated in the beginning of the document).

Setting mail server parameters in config.php -> SMTP -> TLS

grafik

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I merged the SSL and SMTP sections because it doesn't make sense anymore to have them separated. I also revived the paragraph about authentication which I recommended to delete earlier 🙈

grafik

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.

Document new Symfony Mailer behaviour

3 participants