Fix condition check for MANUAL_MODE variable

README.md

@@ -309,7 +309,7 @@ All these variables impact in configuration directives in the modsecurity engine
 | DETECTION_PARANOIA | (:new: Replaces `EXECUTING_PARANOIA` in CRSv4) An integer indicating the detection_paranoia_level (Default: `BLOCKING_PARANOIA`) |
 | ENFORCE_BODYPROC_URLENCODED | A boolean indicating the enforce_bodyproc_urlencoded (Default: `0`) |
 | EXECUTING_PARANOIA | An integer indicating the executing_paranoia_level (Default: `PARANOIA`) |
-| MANUAL_MODE | A boolean indicating that you are providing your own `crs-setup.conf` file mounted as volume. (Default: `0`). ⚠️ None of the following variables are used if you set it to `1`. |
+| MANUAL_MODE | A boolean indicating that you are providing your own crs-setup.conf file mounted as volume. When set to `true`, manual configuration mode is enabled and all automatic CRS configuration via environment variables will be disabled. (Default: `false`). ⚠️ None of the following variables are used if you set it to true. |
 | MAX_FILE_SIZE | An integer indicating the max_file_size (Default: `unlimited`) |
 | MAX_NUM_ARGS | An integer indicating the max_num_args (Default: `unlimited`) |
 | PARANOIA | An integer from `1` through `4`, indicating the paranoia level (Default: `1`) |

src/opt/modsecurity/activate-plugins.sh

@@ -3,9 +3,10 @@
 printf "# # #\nRunning CRS plugin activation\n- - -\n\n"
 
 # Check if crs-setup.conf is overriden
-if [ -n "${MANUAL_MODE}" ]; then
+if [ "${MANUAL_MODE}" = "true" ]; then
   echo "Using manual config mode"
-  return; # Don't use exit on a sourced script
+  # Don't use exit on a sourced script
+  return
 fi
 
 # Plugins can be disabled entirely by setting CRS_DISABLE_PLUGINS.

src/opt/modsecurity/configure-rules.sh

@@ -11,7 +11,7 @@ printf "# # #\nRunning CRS rule configuration\n- - -\n"
 DIRECTORY="$(cd "$(dirname "$0")" && pwd)"
 
 # Check if crs-setup.conf is overriden
-if [ -n "${MANUAL_MODE}" ]; then
+if [ "${MANUAL_MODE}" = "true" ]; then
   echo "Using manual config mode"
   # Don't use exit on a sourced script
   return