refactor: replace unnecessary use of default_factory with default (…

…#2923) * refactor: fix misuse of default_factory from dataclass field * refactor: set middleware_class as field like other fields Co-authored-by: Boseong Choi <[email protected]> --------- Co-authored-by: Boseong Choi <[email protected]> Co-authored-by: Peter Schutt <[email protected]>
litestar-org · Dec 23, 2023 · 02ca07d · 02ca07d
1 parent 91762e5
commit 02ca07d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/litestar/middleware/logging.py b/litestar/middleware/logging.py
@@ -271,7 +271,7 @@ class LoggingMiddlewareConfig:
     response_log_message: str = field(default="HTTP Response")
     """Log message to prepend when logging a response."""
     request_log_fields: Iterable[RequestExtractorField] = field(
-        default_factory=lambda: (
+        default=(
             "path",
             "method",
             "content_type",
@@ -290,7 +290,7 @@ class LoggingMiddlewareConfig:
         -  To turn off logging of requests, use and empty iterable.
     """
     response_log_fields: Iterable[ResponseExtractorField] = field(
-        default_factory=lambda: (
+        default=(
             "status_code",
             "cookies",
             "headers",
@@ -305,7 +305,7 @@ class LoggingMiddlewareConfig:
             Thus, re-arranging the log-message is as simple as changing the iterable.
         -  To turn off logging of responses, use and empty iterable.
     """
-    middleware_class: type[LoggingMiddleware] = field(default_factory=lambda: LoggingMiddleware)
+    middleware_class: type[LoggingMiddleware] = field(default=LoggingMiddleware)
     """Middleware class to use.
 
     Should be a subclass of [litestar.middleware.LoggingMiddleware].