From 7bb86e41189682d4c2fb502e672bd1dce99fb3c3 Mon Sep 17 00:00:00 2001
From: Adam Warner <me@adamwarner.co.uk>
Date: Wed, 18 Jan 2023 22:12:29 +0000
Subject: [PATCH] Add a check for the version number of lighttpd. Only remove
 the server.modules += section of the new pihole-admin config if the version
 number is not greater than or equal to 1.4.56

Signed-off-by: Adam Warner <me@adamwarner.co.uk>
---
 advanced/lighttpd.conf.debian      |  1 -
 automated install/basic-install.sh | 19 ++++++++++++++-----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/advanced/lighttpd.conf.debian b/advanced/lighttpd.conf.debian
index 06c284fe..61d20821 100644
--- a/advanced/lighttpd.conf.debian
+++ b/advanced/lighttpd.conf.debian
@@ -17,7 +17,6 @@
 
 server.modules = (
     "mod_access",
-    "mod_accesslog",
     "mod_auth",
     "mod_expire",
     "mod_redirect",
diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh
index 3b37bf28..44b4b313 100755
--- a/automated install/basic-install.sh	
+++ b/automated install/basic-install.sh	
@@ -1432,14 +1432,23 @@ installConfigs() {
         elif [[ -d "/etc/lighttpd/conf-available" ]]; then
             conf=/etc/lighttpd/conf-available/15-pihole-admin.conf
             install -D -m 644 -T ${PI_HOLE_LOCAL_REPO}/advanced/pihole-admin.conf $conf
-            # disable server.modules += ( ... ) in $conf to avoid module dups
-            # (needed until Debian 10 no longer supported by pi-hole)
-            # (server.modules duplication is ignored in lighttpd 1.4.56+)
-            if awk '!/^server\.modules/{print}' $conf > $conf.$$ && mv $conf.$$ $conf; then
+
+            # Get the version number of lighttpd
+            version=$(lighttpd -v | grep -o -E '[0-9]+\.[0-9]+\.[0-9]+')
+            # Test if that version is greater than or euqal to 1.4.56
+            if dpkg --compare-versions "$version" "ge" "1.4.56"; then
+                # If it is, then we don't need to disable the modules
                 :
             else
-                rm $conf.$$
+                # disable server.modules += ( ... ) in $conf to avoid module dups
+                # (server.modules duplication is ignored in lighttpd 1.4.56+)
+                if awk '!/^server\.modules/{print}' $conf > $conf.$$ && mv $conf.$$ $conf; then
+                :
+                else
+                    rm $conf.$$
+                fi
             fi
+
             chmod 644 $conf
             if is_command lighty-enable-mod ; then
                 lighty-enable-mod pihole-admin access accesslog redirect fastcgi setenv > /dev/null || true