docker: Fix using a domain directory path as a domain name

albertito / chasquid

SMTP (email) server with a focus on simplicity, security, and ease of operation [mirror]

https://blitiri.com.ar/p/chasquid/

Other

868 stars 56 forks source link

docker: Fix using a domain directory path as a domain name #42

Closed erjoalgo closed 10 months ago

erjoalgo commented 10 months ago

This causes an invalid config in the generated /etc/dovecot/auto-ssl.conf, e.g.:

ssl_cert = </etc/letsencrypt/live//etc/letsencrypt/live/mail.example.com/fullchain.pem ssl_key = </etc/letsencrypt/live//etc/letsencrypt/live/mail.example.com/privkey.pem

albertito commented 10 months ago

Thank you for finding this bug and sending a patch for it!

I've merged this in the next branch with minor modifications to the commit message: 359d45a3de97e2da1585f35565d4f89102dfb6ea

Does that seem okay to you? If so, I will merge it on the master branch too.

Thanks again!

erjoalgo commented 10 months ago

It seems great to me, thank you for your quick response and your work on this repository.

albertito commented 10 months ago

Thanks to you for finding this and sending a patch!

Merged in both next and master :)