Skip to content

Commit

Permalink
[deploy-vhost] Use matching idiom in calls to make_dir
Browse files Browse the repository at this point in the history
Although `make_dir` will use `$conf->{user_uid}` if it's 3rd argument
is undefined, I think it's clearer to be explicit here, especially as
we're also setting the GID.
  • Loading branch information
sagepe committed Nov 20, 2023
1 parent a46557b commit 1a2fae2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bin/deploy-vhost
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ sub create_or_update_files {
# Allow logs to be read on staging hosts so that tests can run
# Protect logs on production hosts
$conf->{staging} ? 0755 : 0750,
0, scalar(getgrnam('adm')));
$conf->{user_uid}, scalar(getgrnam('adm')));
make_dir("$vhost_dir/applogs", 0750, $conf->{user_uid}, scalar(getgrnam('adm')));
my $web_dir = $conf->{redirects_only} ? "$vhost_dir/web" : "$vcspath/" . $conf->{web_dir};
make_symlink($web_dir, "$vhost_dir/docs");
Expand Down

0 comments on commit 1a2fae2

Please sign in to comment.