Skip to content

docs: clarification of description of explicit TLS#3017

Merged
polarathene merged 4 commits intodocker-mailserver:masterfrom
i-C-o-d-e-r:master
Jan 22, 2023
Merged

docs: clarification of description of explicit TLS#3017
polarathene merged 4 commits intodocker-mailserver:masterfrom
i-C-o-d-e-r:master

Conversation

@i-C-o-d-e-r
Copy link
Copy Markdown
Contributor

@i-C-o-d-e-r i-C-o-d-e-r commented Jan 21, 2023

Description

Changed description of explicit TLS to indicate that insecure connections are rejected
Fixes #3007

Type of change

  • Clarification of description of Explicit TLS

@georglauterbach georglauterbach self-assigned this Jan 21, 2023
@georglauterbach georglauterbach added kind/improvement Improve an existing feature, configuration file or the documentation area/documentation labels Jan 21, 2023
@georglauterbach georglauterbach changed the title Clearification of description of explicit TLS docs: clarification of description of explicit TLS Jan 22, 2023
Copy link
Copy Markdown
Member

@polarathene polarathene left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to improve our documentation! 🙏

I've expanded on your addition a bit more to clarify that it only applies to authentication (to submit mail to send, or to retrieve mail to a MUA), but does not apply to receiving mail from third-parties on port 25 👍

Comment thread docs/content/config/security/understanding-the-ports.md Outdated
@polarathene polarathene added this to the v12.0.0 milestone Jan 22, 2023
Copy link
Copy Markdown
Member

@georglauterbach georglauterbach left a comment

Choose a reason for hiding this comment

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

👍🏼

@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: 92e7675

@polarathene polarathene merged commit b2cd66f into docker-mailserver:master Jan 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation kind/improvement Improve an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear documentation for the description of the port 110

3 participants