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

Support for self signed certificates and using a different domain for email accounts vs internal domain #21

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

paz
Copy link

@paz paz commented Mar 28, 2021

I'm not very good with python, so I'm not sure if my code is very good. I've tested it on my setup and it's working nicely however.

The domain replacement is quite simple, it just replaces the domain if another is defined.
The self signed certs aren't done by just ignoring validation, the container will run update-ca-certificates and you would have to add your certificates to the host and then uncomment the volume.
None of these features are enabled by default, users will have to uncomment them in docker override yml.

@Programmierus
Copy link
Owner

Hi! Thanks for your effort! I am currently damn busy on another project, but deadline is soon. Have placed it on top of my TODO to check your pull request and also other issues with this project.

Thanks again.

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.

2 participants