refactor(logging): standardize log format with additional context

Author: MickaelCa

src/server/metrics_server.py

@@ -53,7 +53,7 @@ def start_metrics_server(host: str = "127.0.0.1", port: int = 9090) -> None:
     None
 
     """
-    logger.info("Starting metrics server on %s:%s", host, port)
+    logger.info("Starting metrics server", extra={"host": host, "port": port})
 
     # Configure uvicorn to suppress startup messages to avoid duplicates
     # since the main server already shows similar messages

src/server/query_processor.py

@@ -114,7 +114,7 @@ async def _check_s3_cache(
             )
     except Exception as exc:
         # Log the exception but don't fail the entire request
-        logger.warning("S3 cache check failed, falling back to normal cloning: %s", exc)
+        logger.warning("S3 cache check failed, falling back to normal cloning", extra={"error": str(exc)})
 
     return None
 
@@ -168,7 +168,7 @@ def _store_digest_content(
             logger.info("Successfully uploaded metadata to S3")
         except Exception as metadata_exc:
             # Log the error but don't fail the entire request
-            logger.warning("Failed to upload metadata to S3: %s", metadata_exc)
+            logger.warning("Failed to upload metadata to S3", extra={"error": str(metadata_exc)})
 
         # Store S3 URL in query for later use
         query.s3_url = s3_url

src/server/s3_utils.py

@@ -371,14 +371,14 @@ def get_metadata_from_s3(s3_file_path: str) -> S3Metadata | None:
         # Object doesn't exist if we get a 404 error
         error_code = err.response.get("Error", {}).get("Code")
         if error_code == "404":
-            logger.info("Metadata file not found: %s", metadata_file_path)
+            logger.info("Metadata file not found", extra={"metadata_file_path": metadata_file_path})
             return None
         # Log other errors but don't fail
-        logger.warning("Failed to retrieve metadata from S3: %s", err)
+        logger.warning("Failed to retrieve metadata from S3", extra={"error": str(err)})
         return None
     except Exception as exc:
         # For any other exception, log and return None
-        logger.warning("Unexpected error retrieving metadata from S3: %s", exc)
+        logger.warning("Unexpected error retrieving metadata from S3", extra={"error": str(exc)})
         return None
 
 
@@ -497,7 +497,7 @@ def get_s3_url_for_ingest_id(ingest_id: UUID) -> str | None:
 
     """
     if not is_s3_enabled():
-        logger.debug("S3 not enabled, skipping URL lookup for ingest_id: %s", ingest_id)
+        logger.debug("S3 not enabled, skipping URL lookup", extra={"ingest_id": str(ingest_id)})
         return None
 
     logger.info("Starting S3 URL lookup for ingest ID", extra={"ingest_id": str(ingest_id)})