velocity-python 0.0.212__tar.gz → 0.0.213__tar.gz

{velocity_python-0.0.212 → velocity_python-0.0.213}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.0.212
+Version: 0.0.213
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.0.212 → velocity_python-0.0.213}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "velocity-python"
-version = "0.0.212"
+version = "0.0.213"
 authors = [
   { name="Velocity Team", email="info@codeclubs.org" },
 ]

{velocity_python-0.0.212 → velocity_python-0.0.213}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.0.212"
+__version__ = version = "0.0.213"
 
 from . import aws
 from . import db

{velocity_python-0.0.212 → velocity_python-0.0.213}/src/velocity/aws/handlers/base_handler.py

@@ -288,12 +288,49 @@ class BaseHandler:
                             "onerror_exc": on_error_exc.__class__.__name__,
                         },
                     )
-                    return
-                raise
-        else:
-            # Re-raise if no error handler is defined
+                else:
+                    raise
+
+        if self._set_unhandled_error_response(local_context):
+            return
+
+        if not hasattr(self, "onError"):
+            # Non-HTTP contexts without a custom error hook should still fail fast.
             raise exception
 
+    def _set_unhandled_error_response(self, local_context) -> bool:
+        response_getter = getattr(local_context, "response", None)
+        if not callable(response_getter):
+            return False
+
+        response = response_getter()
+        if response is None:
+            return False
+
+        reference_id = getattr(self.aws_context, "aws_request_id", None)
+        title = "Request Failed"
+        message = "Something went wrong while processing your request."
+        detail = "More information has been logged on the backend."
+
+        body = {
+            "message": message,
+            "detail": detail,
+            "logged": True,
+        }
+
+        user_message = f"{message} {detail}"
+        console_message = detail
+        if reference_id:
+            body["reference_id"] = str(reference_id)
+            user_message = f"{user_message} Reference: {reference_id}."
+            console_message = f"{console_message} Reference: {reference_id}."
+
+        response.set_status(500)
+        response.set_body(body)
+        response.console(console_message, title=title)
+        response.alert(user_message, title=title)
+        return True
+
     def _is_transient_db_disconnect(self, exc: Exception = None, *args, **kwargs) -> bool:
         """Return True if an exception looks like a transient DB disconnect.
 

velocity_python-0.0.213/src/velocity/aws/tests/test_base_handler_error_response.py

@@ -0,0 +1,62 @@
+import json
+import unittest
+from types import SimpleNamespace
+
+from velocity.aws.handlers.base_handler import BaseHandler
+from velocity.aws.handlers.response import Response
+
+
+class DummyContext:
+    def __init__(self, response):
+        self._response = response
+
+    def response(self):
+        return self._response
+
+    def action(self):
+        return "button-click"
+
+
+class DummyHandler(BaseHandler):
+    def __init__(self, aws_context=None):
+        super().__init__({}, aws_context or SimpleNamespace(aws_request_id="req-123"))
+        self.on_error_calls = []
+
+    def onError(self, tx, context, exc, tb):
+        self.on_error_calls.append({"exc": exc, "tb": tb})
+
+
+class TestBaseHandlerErrorResponse(unittest.TestCase):
+    def test_unhandled_http_error_sets_generic_response(self):
+        handler = DummyHandler()
+        response = Response()
+        context = DummyContext(response)
+
+        handler.handle_error(tx=None, local_context=context, exception=RuntimeError("secret details"))
+
+        self.assertEqual(response.status(), 500)
+        self.assertEqual(response.body["message"], "Something went wrong while processing your request.")
+        self.assertEqual(response.body["detail"], "More information has been logged on the backend.")
+        self.assertTrue(response.body["logged"])
+        self.assertEqual(response.body["reference_id"], "req-123")
+
+        rendered = response.render()
+        body = json.loads(rendered["body"])
+        self.assertEqual(rendered["statusCode"], 500)
+        self.assertEqual(len(body["actions"]), 2)
+        self.assertEqual(body["actions"][0]["action"], "console")
+        self.assertEqual(body["actions"][1]["action"], "alert")
+        self.assertIn("Reference: req-123.", body["actions"][1]["payload"]["message"])
+        self.assertNotIn("secret details", body["actions"][1]["payload"]["message"])
+        self.assertEqual(len(handler.on_error_calls), 1)
+
+    def test_unhandled_non_http_error_without_onerror_reraises(self):
+        handler = BaseHandler({}, SimpleNamespace(aws_request_id="req-456"))
+        context = DummyContext(None)
+
+        with self.assertRaises(RuntimeError):
+            handler.handle_error(tx=None, local_context=context, exception=RuntimeError("boom"))
+
+
+if __name__ == "__main__":
+    unittest.main()

{velocity_python-0.0.212 → velocity_python-0.0.213}/src/velocity_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.0.212
+Version: 0.0.213
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.0.212 → velocity_python-0.0.213}/src/velocity_python.egg-info/SOURCES.txt

@@ -27,6 +27,7 @@ src/velocity/aws/handlers/mixins/__init__.py
 src/velocity/aws/handlers/mixins/data_service.py
 src/velocity/aws/handlers/mixins/web_handler.py
 src/velocity/aws/tests/__init__.py
+src/velocity/aws/tests/test_base_handler_error_response.py
 src/velocity/aws/tests/test_lambda_handler_json_serialization.py
 src/velocity/aws/tests/test_response.py
 src/velocity/db/__init__.py