about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2024-01-30 07:14:07 +0300
committerFrederick Muriuki Muriithi2024-01-30 07:15:07 +0300
commita1741736c5b14af81e398d228e22d85de51d2c4b (patch)
treea713752fe27fec842ac4fa4935640558b04f9f22
parent48ccd489508fcb2f6285561902826025b0c4e74c (diff)
downloadgenenetwork3-a1741736c5b14af81e398d228e22d85de51d2c4b.tar.gz
Add the traceback to the exceptions for display.
-rw-r--r--gn3/errors.py24
1 files changed, 17 insertions, 7 deletions
diff --git a/gn3/errors.py b/gn3/errors.py
index cd707f3..ec0088d 100644
--- a/gn3/errors.py
+++ b/gn3/errors.py
@@ -1,32 +1,42 @@
 """Handle application level errors."""
+import traceback
+
 from sqlite3 import OperationalError
+from werkzeug.exceptions import NotFound
 from flask import Flask, jsonify, current_app
 from authlib.oauth2.rfc6749.errors import OAuth2Error
 
 from gn3.auth.authorisation.errors import AuthorisationError
 
+def add_trace(exc: Exception, jsonmsg: dict) -> dict:
+    """Add the traceback to the error handling object."""
+    return {
+        **jsonmsg,
+        "error-trace": "".join(traceback.format_exception(exc))
+    }
+
 def page_not_found(pnf):
     """Generic 404 handler."""
-    return jsonify({
+    return jsonify(add_trace(pnf, {
         "error": pnf.name,
         "error_description": pnf.description
-    }), 404
+    })), 404
 
 def handle_authorisation_error(exc: AuthorisationError):
     """Handle AuthorisationError if not handled anywhere else."""
     current_app.logger.error(exc)
-    return jsonify({
+    return jsonify(add_trace(exc, {
         "error": type(exc).__name__,
         "error_description": " :: ".join(exc.args)
-    }), exc.error_code
+    })), exc.error_code
 
 def handle_oauth2_errors(exc: OAuth2Error):
     """Handle OAuth2Error if not handled anywhere else."""
     current_app.logger.error(exc)
-    return jsonify({
+    return jsonify(add_trace(exc, {
         "error": exc.error,
         "error_description": exc.description,
-    }), exc.status_code
+    })), exc.status_code
 
 def handle_sqlite3_errors(exc: OperationalError):
     """Handle sqlite3 errors if not handled anywhere else."""
@@ -38,7 +48,7 @@ def handle_sqlite3_errors(exc: OperationalError):
 
 def register_error_handlers(app: Flask):
     """Register application-level error handlers."""
-    app.register_error_handler(404, page_not_found)
+    app.register_error_handler(NotFound, page_not_found)
     app.register_error_handler(OAuth2Error, handle_oauth2_errors)
     app.register_error_handler(OperationalError, handle_sqlite3_errors)
     app.register_error_handler(AuthorisationError, handle_authorisation_error)