velocity-python 0.0.188__tar.gz → 0.0.190__tar.gz

{velocity_python-0.0.188/src/velocity_python.egg-info → velocity_python-0.0.190}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.0.188
+Version: 0.0.190
 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.188 → velocity_python-0.0.190}/pyproject.toml

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

{velocity_python-0.0.188 → velocity_python-0.0.190}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.0.188"
+__version__ = version = "0.0.190"
 
 from . import aws
 from . import db

{velocity_python-0.0.188 → velocity_python-0.0.190}/src/velocity/aws/handlers/context.py

@@ -363,7 +363,7 @@ class Context:
             logger.error("Error getting layer information: %s", exc)
             return []
 
-    def get_pass_through_vars(self, tx, current_user=None):
+    def get_pass_through_vars(self, tx):
         """Return environment and layer information for controls.vars."""
         vars = dict(
             [
@@ -380,14 +380,25 @@ class Context:
             ]
         )
 
+        temp = {"controls": {"vars": vars}}
+
+        # if current_user:
+        #     temp["controls"]["current_user"] = current_user
+
+        return temp
+    
+    def get_logged_in_email(self):
+        """Return the current user information from the session."""
+        return self.session().get("email_address")
+    
+   def get_version_vars(self):
+        """Return environment and layer information for controls.vars."""
+       
         vars["velocity"] = velocity.__version__
         vars["layers"] = self.get_current_lambda_layers()
 
         temp = {"controls": {"vars": vars}}
 
-        if current_user:
-            temp["controls"]["current_user"] = current_user
-
         return temp
 
     def _load_feature_flags(self, tx):

{velocity_python-0.0.188 → velocity_python-0.0.190}/src/velocity/aws/handlers/lambda_handler.py

@@ -251,25 +251,6 @@ class LambdaHandler(BaseHandler):
             except:
                 raise Exception("User Authentication Error [Cognito]")
 
-    def get_pass_through_vars(self, tx, context):
-        """Get pass-through variables: environment vars, layers, and user data"""
-        temp = context.get_pass_through_vars(tx, self.current_user)
-
-        logger.debug(
-            "LAYER_DEBUG: vars object contains: %s",
-            list(temp.get("controls", {}).get("vars", {}).keys()),
-        )
-        if "layers" in temp.get("controls", {}).get("vars", {}):
-            logger.debug(
-                "LAYER_DEBUG: layers value: %s",
-                temp["controls"]["vars"]["layers"],
-            )
-
-        # Merge with user data from database
-        temp = context.merge_user_data(tx, temp, self.session["email_address"])
-
-        return temp
-
     def _summarize_postdata(self, postdata):
         """Extract key fields from postdata for logging without storing entire payload"""
         if not postdata:

{velocity_python-0.0.188 → velocity_python-0.0.190}/src/velocity/aws/handlers/mixins/data_service.py

@@ -8,10 +8,13 @@ that uses velocity.db for database access.
 import base64
 import datetime
 import importlib
+import logging
 from io import BytesIO
 
 from velocity.misc import export
 
+logger = logging.getLogger(__name__)
+
 
 class DataServiceMixin:
     """
@@ -71,6 +74,7 @@ class DataServiceMixin:
             *args: Arguments to pass to the hook
             **kwargs: Keyword arguments to pass to the hook
         """
+        
         try:
             m = importlib.import_module(f".{table}", "rwx")
             if hasattr(m, hook_name):
@@ -275,10 +279,14 @@ class DataServiceMixin:
         if not any(value is not None for value in incoming.values()):
             raise ValueError("Parameter 'object' cannot contain only None values")
 
-        print(
-            f"WRITE_LOG: Writing to table {table_name} with sys_id {incoming.get('sys_id')}"
+        logger.debug(
+            "Writing to table",
+            extra={
+                "table_name": table_name,
+                "sys_id": incoming.get("sys_id"),
+                "object_keys": list(incoming.keys()),
+            },
         )
-        print(f"WRITE_LOG: Object keys: {list(incoming.keys())}")
 
         try:
             row = self.write_hook(
@@ -290,7 +298,10 @@ class DataServiceMixin:
             )
 
             if not row:
-                print(f"WARNING: self.write_hook returned empty row for table {table_name}")
+                logger.warning(
+                    "write_hook returned empty row",
+                    extra={"table_name": table_name},
+                )
                 row = {}
 
             context.response().set_body(
@@ -299,11 +310,20 @@ class DataServiceMixin:
                     "lastFetch": datetime.datetime.now(),
                 }
             )
-            print(f"WRITE_LOG: Successfully wrote to {table_name}")
+            logger.debug(
+                "Successfully wrote to table",
+                extra={"table_name": table_name},
+            )
 
         except Exception as e:
-            print(f"ERROR in OnActionWriteObject: {e}")
-            print(f"table_name={table_name}, incoming_keys={list(incoming.keys())}")
+            logger.error(
+                "Error in OnActionWriteObject",
+                extra={
+                    "exception": str(e),
+                    "table_name": table_name,
+                    "incoming_keys": list(incoming.keys()),
+                },
+            )
             raise
         context.response().load_object(row)
 

{velocity_python-0.0.188 → velocity_python-0.0.190}/src/velocity/aws/handlers/mixins/web_handler.py

@@ -9,6 +9,7 @@ onError implementations in handlers.
 import copy
 import importlib
 import json
+import logging
 import os
 import pprint
 import time
@@ -18,6 +19,8 @@ from typing import Any, Dict, List
 
 from velocity.aws.handlers.exceptions import AlertError
 
+logger = logging.getLogger(__name__)
+
 class WebHandler(ABC):
     """
     Mixin providing unified activity tracking plus standardized error handling.
@@ -72,9 +75,12 @@ class WebHandler(ABC):
         try:
             self.activity_log_key = tx.table("aws_api_activity").new(self.activity_data).pk
         except Exception as exc:
-            context.log(
-                f"WebHandler.track_activity_start failed: {exc}; keys={list(self.activity_data.keys())}",
-                "WebHandler.track_activity_start",
+            logger.error(
+                "WebHandler.track_activity_start failed",
+                extra={
+                    "exception": str(exc),
+                    "activity_data_keys": list(self.activity_data.keys()),
+                },
             )
             raise
 
@@ -254,7 +260,7 @@ Request Details:
             html = html.replace("\n", "<br>").replace(" ", "&nbsp;")
             recipients = self._resolve_error_notification_recipients(tx, context)
             if not recipients:
-                print("No error notification recipients configured; skipping notification.")
+                logger.debug("No error notification recipients configured; skipping notification.")
                 return
 
             sender = self._resolve_error_notification_sender(tx, context)
@@ -271,7 +277,7 @@ Request Details:
                 email_settings_id=1001,
             )
         except Exception as email_error:  # pragma: no cover
-            print(f"Failed to send error notification email: {email_error}")
+            logger.error("Failed to send error notification email", extra={"error": str(email_error)})
 
     def _resolve_error_notification_recipients(self, tx, context) -> List[str]:
         recipients: List[str] = []
@@ -280,7 +286,7 @@ Request Details:
             try:
                 recipients = context_method(tx) or []
             except Exception as exc:
-                print(f"Failed to load recipients from context: {exc}")
+                logger.warning("Failed to load recipients from context", extra={"error": str(exc)})
         if recipients:
             return recipients
         return self._get_error_notification_recipients()
@@ -292,7 +298,7 @@ Request Details:
             try:
                 sender = context_method(tx)
             except Exception as exc:
-                print(f"Failed to load sender from context: {exc}")
+                logger.warning("Failed to load sender from context", extra={"error": str(exc)})
         if sender:
             return sender
         return self._get_error_notification_sender()

{velocity_python-0.0.188 → velocity_python-0.0.190/src/velocity_python.egg-info}/PKG-INFO

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