Remove write access to group and others to Amavis configuration files#1632
Remove write access to group and others to Amavis configuration files#1632erik-wramner merged 2 commits intodocker-mailserver:masterfrom artonge:patch-1
Conversation
Remove write access to group and others to Amavis configuration files Fix: #1446
|
I will merge this, but I want to hear @erik-wramner opinion in #1446 first. |
|
The chmod affects not only the copied files but any other files that happen to be in the folder as well. I would suggest another approach, see moby/moby#34263. It is now possible to set the permissions and owner in the COPY command in the Dockerfile. If we do that we save one layer in the final image and we don't risk changing existing files in a negative way. @artonge, can you try that instead, please? |
georglauterbach
left a comment
georglauterbach
left a comment
There was a problem hiding this comment.
It is now possible to set the permissions and owner in the COPY command in the Dockerfile. If we do that we save one layer in the final image and we don't risk changing existing files in a negative way.
Exactly why I wanted to hear @erik-wramner on this. Thanks :D
|
Hm, chmod will be supported (moby/moby#34819) but is not there yet. I'll merge this then. |
3 participants
Remove write access to group and others to Amavis configuration files
Fix: #1446