Skip to content
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

Setup MTA-STS #75

Merged
merged 13 commits into from
Nov 24, 2023
Merged

Setup MTA-STS #75

merged 13 commits into from
Nov 24, 2023

Conversation

missytake
Copy link
Contributor

scripts/generate-dns-zone.sh Outdated Show resolved Hide resolved
@hpk42
Copy link
Contributor

hpk42 commented Nov 22, 2023 via email

…for now, let's assume A/AAAA point to the chatmail server, too
@missytake
Copy link
Contributor Author

missytake commented Nov 22, 2023

For now, the model is website==mxsite -- i'd prefer this question/issue to stay out of the PR. If we want, then a dedicated separate PR with docs that introduces the possible split between http and mx servers even though i am not in favor of that right now as we are trying to stabilize, not add "todo's" into the code and docs. next week i'd like to go live, resolve existing issues, and get it ready, not introduce TODOs implicitely.

alright, I removed all related changes from this PR. This is only about MTA-STS now.

@link2xt
Copy link
Contributor

link2xt commented Nov 23, 2023

alright, I removed all related changes from this PR. This is only about MTA-STS now.

But nginx config is still changed and has duplicate server entry.

@missytake
Copy link
Contributor Author

missytake commented Nov 23, 2023

alright, I removed all related changes from this PR. This is only about MTA-STS now.

But nginx config is still changed and has duplicate server entry.

yes, because the mta-sts.$CHATMAIL_DOMAIN HTTPS route needs to serve a different TLS certificate than the $CHATMAIL_DOMAIN HTTPS route - one server entry in the nginx config is not enough unfortunately.

deploy-chatmail/src/deploy_chatmail/__init__.py Outdated Show resolved Hide resolved
@missytake missytake merged commit 776bd87 into main Nov 24, 2023
4 checks passed
@missytake missytake deleted the mta-sts branch November 25, 2023 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Try out enabling MTA-STS to secure/enforce TLS connections with compliant providers
3 participants