Process feedback

Author: PGijsbers

src/core/logging.py

@@ -34,7 +34,11 @@ async def add_request_context_to_log(
 ) -> Response:
     """Add a unique request id to each log call."""
     identifier = uuid.uuid4().hex
-    with logger.contextualize(request_id=identifier):
+    with logger.contextualize(
+        request_id=identifier,
+        method=request.method,
+        path=request.url.path,
+    ):
         return await call_next(request)
 
 
@@ -61,6 +65,7 @@ async def log_request_duration(
         process_ms=int(duration_process_ns / 1_000_000),
         wallclock_time_ns=duration_mono_ns,
         process_time_ns=duration_process_ns,
+        status=response.status_code,
     )
     return response
 

src/routers/dependencies.py

@@ -39,7 +39,7 @@ async def fetch_user(
         raise AuthenticationFailedError(msg)
 
     logger.info(
-        "User {identifier} authenticated in with api key ending in '{api_key}'.",
+        "User {identifier} authenticated with api key ending in '{api_key}'.",
         identifier=user.user_id,
         api_key=masked_key,
     )

src/routers/openml/study.py

@@ -75,9 +75,10 @@ async def attach_to_study(
     if study.creator != user.user_id and not await user.is_admin():
         msg = f"Study {study_id} can only be edited by its creator."
         logger.warning(
-            "User attempted to attach entities to study they do not own.",
+            "User {user_id} attempted to attach entities to study they do not own.",
             study_id=study_id,
             entity_ids=entity_ids,
+            user_id=user.user_id,
         )
         raise StudyNotEditableError(msg)
     if study.status != StudyStatus.IN_PREPARATION: