check .pem existence and concat to combined.pem with added eol#118
check .pem existence and concat to combined.pem with added eol#118tomav merged 1 commit intodocker-mailserver:masterfrom willtho89:master
Conversation
Checks existince of cert, chain and privkey.pem. Concats these files to combined.pem and adds a linebreak after each file if there were none. This commit doesn't change the LE created .pem files, so no unlinking on the host should happen (see #111)
|
Thank you @reldeiS |
|
@reldeiS Sorry, me again! The problem is that those
So even the sed command fails with: I think we have to remove the Anyway, as for branch v2, it seems that for dovecot we don't need any more the combined .pem. I'm running my server with this version and the only certs I'm using right now are:
It's the same as for Postfix. |
check .pem existence and concat to combined.pem with added eol. Minor changes to command line that combine certs.
|
I'm on vacation right now, so i have no access to my PC. Could you PR your fix, @00angus? The curious thing is: it worked (and actually works right now) on my system. Quickedit: It could be that my own lets encrypt script created the combined.pem file, but i'd need to check that |
|
@reldeiS PR submitted right now |
Checks existince of cert, chain and privkey.pem.
Concats these files to combined.pem and adds a linebreak after each file
if there were none. This commit doesn't change the LE created .pem
files, so no unlinking on the host should happen (see
#111)