Error handling (#586)

* feature: debug output
for #447

* wip: the main error flow is defined for #299

* tweak: reverse autoload location search

* wip: error handling

* wip: error handling functionality for #299

* feature: categorise buffer output
closes #585

* feature: debug output and catch errors
closes #447

* feature: errors caught and passed to appropriate error pages
closes #299

Author: Greg Bowler

build.default.json

@@ -7,7 +7,17 @@
 		},
 		"execute": {
 			"command": "webpack",
-			"arguments": ["--entry","./script/script.es6", "--output-path", "./www", "--output-filename", "script.js", "--devtool", "source-map", "--mode", "production"]
+			"arguments": ["--entry","./script/script.es6", "--output-path", "./www", "--output-filename", "script.js", "--devtool", "source-map", "--mode", "development"]
+		}
+	},
+
+	"script/*sw.js": {
+		"require": {
+			"vendor/bin/sync": "*"
+		},
+		"execute": {
+			"command": "vendor/bin/sync",
+			"arguments": ["--pattern", "*sw.js", "script", "www/"]
 		}
 	},
 

composer.lock

@@ -8,6 +8,7 @@
  */
 chdir(dirname($_SERVER["DOCUMENT_ROOT"]));
 ini_set("display_errors", "on");
+ini_set("html_errors", "false");
 /**
  * Before any code is executed, return false here if a static file is requested.
  * When running the PHP inbuilt server, this will output the static file.
@@ -26,7 +27,7 @@
  * files exist.
  * @link https://getcomposer.org/doc/00-intro.md
  */
-foreach([__DIR__, dirname($_SERVER["DOCUMENT_ROOT"])] as $dir) {
+foreach([dirname($_SERVER["DOCUMENT_ROOT"]), __DIR__] as $dir) {
 	$autoloadPath = "$dir/vendor/autoload.php";
 	if(file_exists($autoloadPath)) {
 		/** @noinspection PhpIncludeInspection */

go.php

@@ -1,6 +1,7 @@
 <?php
 namespace Gt\WebEngine\Logic;
 
+use Generator;
 use Gt\Routing\Assembly;
 use Gt\Routing\LogicStream\LogicStreamNamespace;
 use Gt\Routing\LogicStream\LogicStreamWrapper;
@@ -17,7 +18,8 @@ public function __construct(
 		}
 	}
 
-	public function invoke(string $name):void {
+	/** @return Generator<string> filename::function() */
+	public function invoke(string $name):Generator {
 		foreach($this->assembly as $file) {
 			$nsProject = (string)(new LogicProjectNamespace(
 				$file,
@@ -36,6 +38,7 @@ public function invoke(string $name):void {
 						$instance,
 						$name
 					);
+					yield "$file::$name()";
 				}
 			}
 			else {
@@ -52,6 +55,7 @@ public function invoke(string $name):void {
 							null,
 							$fnReference
 						);
+						yield "$file::$name()";
 					}
 				}
 			}
@@ -60,6 +64,6 @@ public function invoke(string $name):void {
 
 	private function loadLogicFile(string $file):void {
 		$streamPath = LogicStreamWrapper::STREAM_NAME . "://$file";
-		require($streamPath);
+		require_once($streamPath);
 	}
 }

src/Logic/LogicExecutor.php

@@ -14,7 +14,9 @@
 use Gt\DomTemplate\PlaceholderBinder;
 use Gt\DomTemplate\TableBinder;
 use Gt\DomTemplate\TemplateCollection;
+use Gt\Http\Header\ResponseHeaders;
 use Gt\Http\Request;
+use Gt\Http\Response;
 use Gt\Http\Uri;
 use Gt\ServiceContainer\Container;
 use Gt\ServiceContainer\LazyLoad;
@@ -26,6 +28,12 @@ public function __construct(
 	) {
 	}
 
+	#[LazyLoad]
+	public function loadResponseHeaders():ResponseHeaders {
+		$response = $this->container->get(Response::class);
+		return $response->headers;
+	}
+
 	#[LazyLoad]
 	public function loadDatabase():Database {
 		$dbSettings = new Settings(

src/Middleware/DefaultServiceLoader.php

@@ -0,0 +1,43 @@
+<?php
+namespace Gt\WebEngine\Middleware;
+
+use Gt\Config\Config;
+use Gt\Dom\HTMLDocument;
+use Gt\DomTemplate\DocumentBinder;
+use Gt\Http\ResponseStatusException\ClientError\ClientErrorException;
+use Gt\Http\ResponseStatusException\ResponseStatusException;
+use Gt\Http\Uri;
+use Gt\ServiceContainer\Container;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+use Throwable;
+
+class ErrorRequestHandler extends RequestHandler {
+	public function __construct(
+		Config $config,
+		callable $finishCallback,
+		callable $obCallback,
+		private Throwable $throwable,
+		protected Container $serviceContainer,
+	) {
+		parent::__construct($config, $finishCallback, $obCallback);
+	}
+
+	public function handle(
+		ServerRequestInterface $request
+	):ResponseInterface {
+		$errorCode = 500;
+		/** @noinspection PhpConditionAlreadyCheckedInspection */
+		if($this->throwable instanceof ResponseStatusException
+		|| $this->throwable instanceof ClientErrorException) {
+			$errorCode = $this->throwable->getHttpCode();
+		}
+
+		$errorUri = new Uri("/_error/$errorCode/");
+		$errorRequest = $request->withUri($errorUri);
+
+		$this->completeRequestHandling($errorRequest, $this->serviceContainer);
+		$this->response = $this->response->withStatus($errorCode);
+		return $this->response;
+	}
+}