Fist attempt at a pluggable error reporting

examples/misc.py

@@ -4,9 +4,18 @@
 from trinket import Trinket, Response
 from trinket.response import file_iterator
 from trinket.extensions import logger
+from trinket.http import HTTPError
 
 
-bauble = logger(Trinket())
+class MyTrinket(Trinket):
+
+    async def on_error(self, http_code, message):
+        if http_code == 404:
+            message = "This route does not exist, i'm so sorry."
+        return await super().on_error(http_code, message)
+
+
+bauble = logger(MyTrinket())
 
 
 @bauble.route('/')

src/trinket/app.py

@@ -5,7 +5,7 @@
 from autoroutes import Routes
 
 from trinket.handler import request_handler
-from trinket.http import HTTPStatus, HTTPError
+from trinket.http import HTTPCode, HTTPStatus, HTTPError
 from trinket.lifecycle import handler_events
 from trinket.proto import Application
 from trinket.request import Request
@@ -25,21 +25,28 @@ def __init__(self):
         self.websockets = set()
         self.hooks = defaultdict(list)
 
+    async def on_error(self, http_code: HTTPCode, message: str=None):
+        """This async method allows the customization of errors.
+        If you need an async log or report or whatever, this is the
+        place to do it.
+        """
+        raise HTTPError(http_code, message)
+
     async def lookup(self, request: Request):
         payload, params = self.routes.match(request.path)
 
         if not payload:
-            raise HTTPError(HTTPStatus.NOT_FOUND, request.path)
+            return await self.on_error(404, request.path)
 
         # Uppercased in order to only consider HTTP verbs.
         handler = payload.get(request.method.upper(), None)
         if handler is None:
-            raise HTTPError(HTTPStatus.METHOD_NOT_ALLOWED)
+            return await self.on_error(HTTPStatus.METHOD_NOT_ALLOWED)
 
         # We check if the route is for a websocket handler.
         # If it is, we make sure we were asked for an upgrade.
         if payload.get('websocket', False) and not request.upgrade:
-            raise HTTPError(
+            return await self.on_error(
                 HTTPStatus.UPGRADE_REQUIRED,
                 'This is a websocket endpoint, please upgrade.')