Skip to content

Commit

Permalink
acme: Use chown -R for challenges directory. Fixes #24529.
Browse files Browse the repository at this point in the history
Commit 75f131d added
`chown 'nginx:nginx' '/var/lib/acme'` to the pre-start script,
but since it doesn't use `chown -R`, it is possible that there
are older existing subdirs (like `acme-challenge`)
that are owned to `root` from before that commit went it.
  • Loading branch information
nh2 committed Apr 1, 2017
1 parent b78f16b commit ee0f3e7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion nixos/modules/security/acme.nix
Expand Up @@ -178,7 +178,7 @@ in
path = [ pkgs.simp_le ];
preStart = ''
mkdir -p '${cfg.directory}'
chown '${data.user}:${data.group}' '${cfg.directory}'
chown -R '${data.user}:${data.group}' '${cfg.directory}'
if [ ! -d '${cpath}' ]; then
mkdir '${cpath}'
fi
Expand Down

0 comments on commit ee0f3e7

Please sign in to comment.