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

Added config folder for Postfix #31

Merged
merged 4 commits into from Aug 2, 2016
Merged

Added config folder for Postfix #31

merged 4 commits into from Aug 2, 2016

Conversation

jkarlosb
Copy link
Contributor

@jkarlosb jkarlosb commented Aug 1, 2016

Feature #29

@jkarlosb
Copy link
Contributor Author

jkarlosb commented Aug 1, 2016

First, I created a volume with postfix config folder. Second, Into postifix container start.sh I added a script to verify optional main.cf and if it exist, override the entries through postconf e flag. Next, the script checks if some .map file exist (e.g. transport.map, sasl_passwd.map, etc.), it copies to /etc/postfix folder, executes postmap and configures its perms. Example config files for basic SES config:

/freeposte/config/postfix/main.cf:

relayhost = [email-smtp.eu-west-1.amazonaws.com]:25
smtp_sasl_auth_enable = yes
smtp_sasl_security_options = noanonymous
smtp_sasl_password_maps = hash:/etc/postfix/sasl_passwd.map
smtp_use_tls = yes
smtp_tls_security_level = encrypt
smtp_tls_note_starttls_offer = yes
smtp_tls_CAfile = /etc/ssl/certs/ca-certificates.crt

/freeposte/config/postfix/sasl_passwd.map:

[email-smtp.eu-west-1.amazonaws.com]:25 USER:PASSWORD

@kaiyou
Copy link
Member

kaiyou commented Aug 1, 2016

Sounds very good.

Because we have very few configuration files for each service, I am thinking about gathering them all in something like /freeposte/overrides (or anything that clearly states that these are overrides, not the actual configuration files) and overriding on a filename basis with proper documentation to avoid having too many directories and different mount points.

The documentation could state something like "create a main.cf to override the postfix configuration", and maybe use some more meaningful names like postfix.conf, dovecot.conf, etc.

The only problem I see would be confidentiality between containers for password maps for instance. Does not sound like too much of a risk. What do you think?

@jkarlosb
Copy link
Contributor Author

jkarlosb commented Aug 2, 2016

Ok, It seems perfect for me, to avoid having too many mount points. About the confidentiality between containers I think this isn't a risk for now.

@kaiyou
Copy link
Member

kaiyou commented Aug 2, 2016

Sounds perfect. Let's go for master, I will test it tonight. Thanks!

@kaiyou kaiyou merged commit 44c1d42 into Mailu:master Aug 2, 2016
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.

None yet

2 participants