setup error handler to make (undefined array key) fatal

defaults/config.ini.default

@@ -17,6 +17,7 @@ allow_die = true ; internal use only
 enable_verbose_error_log = true ; internal use only
 enable_error_to_user = true ; internal use only
 enable_exception_handler = true ; internal use only
+enable_error_handler = true ; internal use only
 
 [ldap]
 uri = "ldap://identity"  ; URI of remote LDAP server

deployment/overrides/phpunit/config/config.ini

@@ -6,3 +6,4 @@ allow_die = false
 enable_verbose_error_log = false
 enable_error_to_user = false
 enable_exception_handler = false
+enable_error_handler = false

resources/init.php

@@ -17,6 +17,10 @@
     set_exception_handler(["UnityWebPortal\lib\UnityHTTPD", "exceptionHandler"]);
 }
 
+if (CONFIG["site"]["enable_error_handler"]) {
+    set_error_handler(["UnityWebPortal\lib\UnityHTTPD", "errorHandler"]);
+}
+
 session_start();
 
 if (isset($GLOBALS["ldapconn"])) {

resources/lib/UnityHTTPD.php

@@ -52,17 +52,17 @@ public static function errorLog(
                 $output["data"] = "data could not be JSON encoded: " . $e->getMessage();
             }
         }
-        $output["REMOTE_USER"] = $_SERVER["REMOTE_USER"] ?? null;
-        $output["REMOTE_ADDR"] = $_SERVER["REMOTE_ADDR"] ?? null;
-        if (!is_null($errorid)) {
-            $output["errorid"] = $errorid;
-        }
         if (!is_null($error)) {
             $output["error"] = self::throwableToArray($error);
         } else {
             // newlines are bad for error log, but getTrace() is too verbose
             $output["trace"] = explode("\n", (new \Exception())->getTraceAsString());
         }
+        $output["REMOTE_USER"] = $_SERVER["REMOTE_USER"] ?? null;
+        $output["REMOTE_ADDR"] = $_SERVER["REMOTE_ADDR"] ?? null;
+        if (!is_null($errorid)) {
+            $output["errorid"] = $errorid;
+        }
         error_log("$title: " . \jsonEncode($output));
     }
 
@@ -132,6 +132,11 @@ public static function internalServerError(
         $errorid = uniqid();
         self::errorToUser("An internal server error has occurred.", 500, $errorid);
         self::errorLog("internal server error", $message, $errorid, $error, $data);
+        if (ini_get("display_errors") && ini_get("html_errors")) {
+            echo "<table>";
+            echo $error->xdebug_message;
+            echo "</table>";
+        }
         self::die($message);
     }
 
@@ -140,7 +145,14 @@ public static function exceptionHandler(\Throwable $e): void
     {
         ini_set("log_errors", true); // in case something goes wrong and error is not logged
         self::internalServerError("An internal server error has occurred.", error: $e);
-        ini_set("log_errors", false); // error logged successfully
+    }
+
+    public static function errorHandler(int $severity, string $message, string $file, int $line)
+    {
+        if (str_contains($message, "Undefined array key")) {
+            throw new ArrayKeyException($message);
+        }
+        return false;
     }
 
     public static function getPostData(...$keys): mixed

tools/docker-dev/web/Dockerfile

@@ -29,7 +29,8 @@ RUN sed -i '/zend.assertions/c\zend.assertions = 1' /etc/php/8.3/cli/php.ini
 RUN sed -i '/memory_limit/c\memory_limit = -1' /etc/php/8.3/apache2/php.ini
 RUN sed -i '/zend.exception_ignore_args/c\zend.exception_ignore_args = 0' /etc/php/8.3/apache2/php.ini
 RUN sed -i '/zend.exception_ignore_args/c\zend.exception_ignore_args = 0' /etc/php/8.3/cli/php.ini
-RUN echo 'xdebug.mode=coverage' >> /etc/php/8.3/cli/php.ini
+RUN echo 'xdebug.mode=develop,coverage,debug,gcstats,profile,trace' >> /etc/php/8.3/cli/php.ini
+RUN echo 'xdebug.mode=develop,coverage,debug,gcstats,profile,trace' >> /etc/php/8.3/apache2/php.ini
 
 # Start apache2 server
 EXPOSE 80