Renamed entrypoint script and configured better log output configuration (Fixes #463)

Author: jaydrogers

docs/content/docs/2.getting-started/3.default-configurations.md

@@ -102,7 +102,7 @@ We also provide a few default scripts to help you get started.
 | **Script Name** | **Description** | **Image variations** |
 |------------|-----------------|----------------------|
 | `0-container-info.sh` | Shows basic execution information, such as Docker User, UID, GID, etc. | all |
-| `1-debug-mode.sh` | Sets PHP to debug mode if `LOG_OUTPUT_LEVEL = "debug"` | all |
+| `1-log-output-level.sh` | Sets PHP log output level to match `LOG_OUTPUT_LEVEL` | all |
 | `10-init-unit.sh` | Processes Unit templates, configures SSL (if enabled), and prepares NGINX Unit for launch | unit |
 | `10-init-webserver-config.sh` | Processes web server configuration templates, configures SSL (if enabled), and prepares web server for launch | *-nginx <br/> *-apache |
 | `50-laravel-automations.sh` | If `AUTORUN_ENABLED` is set to true, and a Laravel installation is detected, the following commands will automatically execute on container start: <br/> - `php artisan config:cache` <br/> - `php artisan route:cache` <br/> - `php artisan view:cache` <br/> - `php artisan event:cache` <br/> - `php artisan migrate --force` | all |

src/common/etc/entrypoint.d/1-debug-mode.sh src/common/etc/entrypoint.d/1-log-output-level.sh

@@ -1,5 +1,5 @@
 #!/bin/sh
-script_name="debug-mode"
+script_name="log-output-level"
 
 if [ "$DISABLE_DEFAULT_CONFIG" = true ]; then
     if [ "$LOG_OUTPUT_LEVEL" = "debug" ]; then
@@ -42,12 +42,19 @@ set_fpm_log_level (){
     fi
 
     fpm_log_level=$1
-    sed -i "/\[global\]/a log_level = $fpm_log_level" /usr/local/etc/php-fpm.conf
+    
+    # Create a temporary file then move it, instead of requiring permissions to write to /usr/local/etc/
+    tmp_file=$(mktemp /tmp/php-fpm.conf.XXXXXX)
+    sed "/\[global\]/a log_level = $fpm_log_level" /usr/local/etc/php-fpm.conf > "$tmp_file"
+    cat "$tmp_file" > /usr/local/etc/php-fpm.conf
+    rm "$tmp_file"
     echo "ℹ️ NOTICE ($script_name): FPM - log_level has been set to \"$fpm_log_level\""
 
-    echo "access.log = /proc/self/fd/2" >> /usr/local/etc/php-fpm.d/zzz-docker-php-serversideup-fpm-debug.conf
-    echo "access.format = \"fpm: %R - %u %t \"%m %r%Q%q\" %s duration=%{milliseconds}dms memory=%Mk cpu=%C%% pid=%p script=%f\"" >> /usr/local/etc/php-fpm.d/zzz-docker-php-serversideup-fpm-debug.conf
-    echo "ℹ️ NOTICE ($script_name): FPM - access.log has been set to \"STDERR\""
+    if [ "$fpm_log_level" = "debug" ]; then 
+        echo "access.log = /proc/self/fd/2" >> /usr/local/etc/php-fpm.d/zzz-docker-php-serversideup-fpm-debug.conf
+        echo "access.format = \"fpm: %R - %u %t \"%m %r%Q%q\" %s duration=%{milliseconds}dms memory=%Mk cpu=%C%% pid=%p script=%f\"" >> /usr/local/etc/php-fpm.d/zzz-docker-php-serversideup-fpm-debug.conf
+        echo "ℹ️ NOTICE ($script_name): FPM - access.log has been set to \"STDERR\""
+    fi
 }
 
 #######################################
@@ -83,7 +90,7 @@ case "$LOG_OUTPUT_LEVEL" in
     set_fpm_log_level alert
     ;;
     *)
-    echo "👉 $script_name: LOG_OUTPUT_LEVEL is not set to a valid value. Please set it to one of the following: debug, info, notice, warn, error, crit, alert, emerg."
+    echo "❌ ERROR ($script_name): LOG_OUTPUT_LEVEL is not set to a valid value. Please set it to one of the following: debug, info, notice, warn, error, crit, alert, emerg."
     return 1
     ;;
 esac