mirror of
https://github.com/pi-hole/pi-hole.git
synced 2024-12-26 14:50:17 +00:00
Apply suggestions from code review
Co-authored-by: DL6ER <dl6er@dl6er.de> Signed-off-by: Christian König <ckoenig@posteo.de>
This commit is contained in:
parent
7bb0ca59d1
commit
3c693c1da5
2 changed files with 3 additions and 3 deletions
|
@ -19,8 +19,8 @@ mkdir -pm 0755 /var/log/pihole
|
||||||
# Ensure that permissions are set so that pihole-FTL can edit all necessary files
|
# Ensure that permissions are set so that pihole-FTL can edit all necessary files
|
||||||
chown -R pihole:pihole /etc/pihole /var/log/pihole
|
chown -R pihole:pihole /etc/pihole /var/log/pihole
|
||||||
chmod -R 0640 /var/log/pihole
|
chmod -R 0640 /var/log/pihole
|
||||||
chmod -R 0660 /etc/pihole /run/pihole
|
chmod -R 0660 /etc/pihole
|
||||||
# allow all users to enter der directories
|
# allow all users to enter the directories
|
||||||
chmod 0755 /etc/pihole /var/log/pihole
|
chmod 0755 /etc/pihole /var/log/pihole
|
||||||
|
|
||||||
# Backward compatibility for user-scripts that still expect log files in /var/log instead of /var/log/pihole
|
# Backward compatibility for user-scripts that still expect log files in /var/log instead of /var/log/pihole
|
||||||
|
|
|
@ -1094,7 +1094,7 @@ installConfigs() {
|
||||||
|
|
||||||
# Install empty custom.list file if it does not exist
|
# Install empty custom.list file if it does not exist
|
||||||
if [[ ! -r "${PI_HOLE_CONFIG_DIR}/custom.list" ]]; then
|
if [[ ! -r "${PI_HOLE_CONFIG_DIR}/custom.list" ]]; then
|
||||||
if ! install -o pihole -g pihole -m 644 /dev/null "${PI_HOLE_CONFIG_DIR}/custom.list" &>/dev/null; then
|
if ! install -o pihole -g pihole -m 660 /dev/null "${PI_HOLE_CONFIG_DIR}/custom.list" &>/dev/null; then
|
||||||
printf " %b Error: Unable to initialize configuration file %s/custom.list\\n" "${COL_LIGHT_RED}" "${PI_HOLE_CONFIG_DIR}"
|
printf " %b Error: Unable to initialize configuration file %s/custom.list\\n" "${COL_LIGHT_RED}" "${PI_HOLE_CONFIG_DIR}"
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
Loading…
Reference in a new issue