Fix permission for *.etag files after gravity run

Signed-off-by: Christian König <github@yubiuser.dev>
Remove readonly from list.sh to avoid errors (#6349 )
2025-07-16 21:00:05 +02:00 · 2025-07-14 21:10:53 -07:00 · 2025-07-14 21:20:39 -03:00 · 2025-07-14 14:45:31 -07:00
2 changed files with 7 additions and 3 deletions
@@ -9,12 +9,12 @@
 # This file is copyright under the latest version of the EUPL.
 # Please see LICENSE file for your rights under this license.

-readonly PI_HOLE_SCRIPT_DIR="/opt/pihole"
-readonly utilsfile="${PI_HOLE_SCRIPT_DIR}/utils.sh"
+PI_HOLE_SCRIPT_DIR="/opt/pihole"
+utilsfile="${PI_HOLE_SCRIPT_DIR}/utils.sh"
 # shellcheck source="./advanced/Scripts/utils.sh"
 source "${utilsfile}"

-readonly apifile="${PI_HOLE_SCRIPT_DIR}/api.sh"
+apifile="${PI_HOLE_SCRIPT_DIR}/api.sh"
 # shellcheck source="./advanced/Scripts/api.sh"
 source "${apifile}"

@@ -808,6 +808,10 @@ gravity_DownloadBlocklistFromUrl() {
      fix_owner_permissions "${saveLocation}"
      # Compare lists if they are identical
      compareLists "${adlistID}" "${saveLocation}"
+      # Set permissions for the *.etag file
+      if [[ -f "${saveLocation}.etag" ]]; then
+          fix_owner_permissions "${saveLocation}.etag"
+      fi
      # Add domains to database table file
      pihole-FTL "${gravity_type}" parseList "${saveLocation}" "${gravityTEMPfile}" "${adlistID}"
      done="true"
Author	SHA1	Message	Date
Christian König	95021fce58	Fix permission for *.etag files after gravity run CodeQL / Analyze (pull_request) Has been cancelled Details Test Supported Distributions / smoke-tests (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (centos_10) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (centos_9) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (debian_11) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (debian_12) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (fedora_40) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (fedora_41) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (fedora_42) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (ubuntu_20) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (ubuntu_22) (pull_request) Has been cancelled Details Test Supported Distributions / distro-test (ubuntu_24) (pull_request) Has been cancelled Details Signed-off-by: Christian König <github@yubiuser.dev>	2025-07-16 21:00:05 +02:00
Dan Schaper	ae048201a2	Remove `readonly` from list.sh to avoid errors (#6349 )	2025-07-14 21:10:53 -07:00
RD WebDesign	dad27f8d06	Remove `readonly` from list.sh to avoid errors Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>	2025-07-14 21:20:39 -03:00
Dan Schaper	c0b512be60	Sync master back into development (#6348 )	2025-07-14 14:45:31 -07:00