Merge pull request #3038 from fractal-analytics-platform/millisec

Simplify middleware timestamp formatting

Author: tcompa

CHANGELOG.md

@@ -17,7 +17,7 @@ The main contents of this release are the introduction of the project sharing an
     * Modify `GET /auth/current-user/allowed-viewer-paths/` logic, with `include_shared_projects` query parameter (\#3031).
     * Add validator for paths to forbid parent-directory references (\#3031).
 * App:
-    * Add `SlowResponseMiddleware` middleware (\#3035).
+    * Add `SlowResponseMiddleware` middleware (\#3035, \#3038).
 * Settings:
     * Add `Settings.FRACTAL_LONG_REQUEST_TIME` configuration variable (\#3035).
 * Database:

fractal_server/main.py

@@ -134,8 +134,6 @@ async def lifespan(app: FastAPI):
 
 slow_response_logger = set_logger("slow-response")
 
-MIDDLEWARE_DATETIME_FORMAT = "%Y-%m-%d %H:%M:%S.%f"
-
 
 class SlowResponseMiddleware:
     def __init__(self, app: FastAPI, time_threshold: float):
@@ -171,8 +169,8 @@ async def send_wrapper(message: Message):
                 f"?{scope['query_string'].decode('utf-8')}, "
                 f"{context['status_code']}, "
                 f"{request_time:.2f}, "
-                f"{start_timestamp.strftime(MIDDLEWARE_DATETIME_FORMAT)}, "
-                f"{end_timestamp.strftime(MIDDLEWARE_DATETIME_FORMAT)}"
+                f"{start_timestamp.isoformat(timespec='milliseconds')}, "
+                f"{end_timestamp.isoformat(timespec='milliseconds')}"
             )