aboutsummaryrefslogtreecommitdiff
path: root/gn3
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 /gn3
parent48ccd489508fcb2f6285561902826025b0c4e74c (diff)
downloadgenenetwork3-a1741736c5b14af81e398d228e22d85de51d2c4b.tar.gz
Add the traceback to the exceptions for display.
Diffstat (limited to 'gn3')
-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)