NRL-2060 Realign handler logs

anjalitrace2-nhs · anjalitrace2-nhs · commit ec713de9d7fd · 2026-03-18T15:31:05.000Z
diff --git a/layer/nrlf/core/decorators.py b/layer/nrlf/core/decorators.py
@@ -156,9 +156,11 @@ def _use_v2_permissions_model(headers: Dict[str, str]) -> bool:
 
 def _load_v2_connection_metadata(headers: Dict[str, str], path: str):
     logger.log(LogReference.HANDLER004d)
+
     metadata = parse_headers(headers, use_v2_permissions=True)
+    logger.log(LogReference.HANDLER003, metadata=metadata.model_dump())
 
-    logger.log(LogReference.HANDLER004e)
+    logger.log(LogReference.HANDLER004b)
     pointer_permissions = get_pointer_permissions_v2(metadata, path)
 
     metadata.nrl_permissions_policy = PermissionsPolicy.model_validate(
@@ -169,10 +171,11 @@ def _load_v2_connection_metadata(headers: Dict[str, str], path: str):
         AccessControls.ALLOW_ALL_TYPES.value
         in metadata.nrl_permissions_policy.access_controls
     ):
+        logger.log(LogReference.HANDLER004a)
         metadata.nrl_permissions_policy.types = PointerTypes.list()
 
     logger.log(
-        LogReference.HANDLER004f,
+        LogReference.HANDLER004c,
         permissions_policy=(
             metadata.nrl_permissions_policy.model_dump()
             if metadata.nrl_permissions_policy
@@ -184,13 +187,16 @@ def _load_v2_connection_metadata(headers: Dict[str, str], path: str):
 
 
 def load_connection_metadata(headers: Dict[str, str], config: Config, path=""):
+    logger.log(LogReference.HANDLER002, headers=headers)
 
     if _use_v2_permissions_model(headers):
         return _load_v2_connection_metadata(headers, path)
 
     metadata = parse_headers(headers, use_v2_permissions=False)
+    logger.log(LogReference.HANDLER003, metadata=metadata.model_dump())
+
     if PERMISSION_ALLOW_ALL_POINTER_TYPES in metadata.nrl_permissions:
-        logger.log(LogReference.HANDLER004b)
+        logger.log(LogReference.HANDLER004a)
         metadata.pointer_types = PointerTypes.list()
         return metadata
 
diff --git a/layer/nrlf/core/log_references.py b/layer/nrlf/core/log_references.py
@@ -30,11 +30,9 @@ class LogReference(Enum):
     )
     HANDLER004 = _Reference("INFO", "Authorisation lookup enabled")
     HANDLER004a = _Reference("INFO", "Authorisation lookup skipped for sync request")
-    HANDLER004b = _Reference("INFO", "Parsing embedded permissions file from S3")
-    HANDLER004c = _Reference("INFO", "Parsed embedded permissions file from S3")
+    HANDLER004b = _Reference("INFO", "Parsing embedded permissions file")
+    HANDLER004c = _Reference("INFO", "Parsed embedded permissions file")
     HANDLER004d = _Reference("INFO", "Using v2 permissions model")
-    HANDLER004e = _Reference("INFO", "Parsing v2 permissions file from lambda layer")
-    HANDLER004f = _Reference("INFO", "Parsed v2 permissions file from lambda layer")
     HANDLER005 = _Reference("WARN", "Rejecting request due to missing pointer types")
     HANDLER006 = _Reference("DEBUG", "Attempting to parse request parameters")
     HANDLER007 = _Reference("INFO", "Parsed request parameters")
