fix: give friendlier error messages during updates #633

Before, access REST API resources resulted in a PHP error traceback which could cause unnecessary concern. This change will allow both endpoints and forms to give a friendlier, more informative notice.

Author: jaredhendrickson13

pfSense-pkg-RESTAPI/files/usr/local/pkg/RESTAPI/Core/Endpoint.inc

@@ -1199,9 +1199,18 @@ class Endpoint {
         $nq_class_name = $this->get_class_shortname();
 
         # Specify the PHP code to write to the Endpoints index.php file
+        $unavailable_error = new ServiceUnavailableError(
+            message: 'This resource is either not installed or is currently updating. Please try again later.',
+            response_id: 'ENDPOINT_UNAVAILABLE',
+        );
+        $unavailable_error_json = json_encode($unavailable_error->to_representation());
         $code =
             "<?php\n" .
-            "require_once('RESTAPI/Endpoints/$nq_class_name.inc');\n" .
+            "\$include = include('RESTAPI/Endpoints/$nq_class_name.inc');\n" .
+            "if (!\$include) {\n" .
+            "    echo '$unavailable_error_json';\n" .
+            "    exit(503);\n" .
+            "}\n" .
             "echo (new $fq_class_name())->process_request();\n" .
             "header('Referer: no-referrer');\n" .
             "session_destroy();\n" .

pfSense-pkg-RESTAPI/files/usr/local/pkg/RESTAPI/Core/Form.inc

@@ -529,14 +529,15 @@ class Form {
 
         # Specify the PHP code to write to the Endpoints index.php file
         $code =
-            "<?php
-            require_once('RESTAPI/Forms/" .
-            $nq_class_name .
-            ".inc');
-            require_once('guiconfig.inc');
-            (new " .
-            $fq_class_name .
-            '())->print_form();';
+            "<?php\n" .
+            "require_once('guiconfig.inc');\n" .
+            "\$include = include('RESTAPI/Forms/$nq_class_name.inc');\n" .
+            "if (!\$include) {\n" .
+            "    echo '<h1>Service Unavailable</h1>';\n" .
+            "    echo 'This resource is either not installed or is currently updating. Please try again later.';\n" .
+            "    exit(503);\n" .
+            "}\n" .
+            "(new $fq_class_name())->print_form();\n";
 
         # Assign the absolute path to the file. Assume index.php filename if not specified.
         $filename = "/usr/local/www/$this->url";