Refactor logging config

server/config/settings/base.py

@@ -304,6 +304,7 @@ else:
         structlog.stdlib.add_logger_name,
         structlog_add_app_info,
         structlog.processors.TimeStamper(fmt="iso"),
+        structlog.processors.CallsiteParameterAdder(),
     ]
 
     LOGGING = {
@@ -314,7 +315,7 @@ else:
                 "()": structlog.stdlib.ProcessorFormatter,
                 "processors": [
                     structlog.stdlib.ProcessorFormatter.remove_processors_meta,
-                    structlog.processors.JSONRenderer()
+                    structlog.processors.JSONRenderer(),
                 ],
                 "foreign_pre_chain": shared_processors,
             },
@@ -350,14 +351,17 @@ else:
             },
             "sentry_sdk": {
                 "level": "ERROR",
-                "handlers": ["console"],
+                "handlers": ["console", "file"],
                 "propagate": False,
             },
         },
     }
 
     structlog.configure(
-        processors=shared_processors + [structlog.stdlib.ProcessorFormatter.wrap_for_formatter,],
+        processors=shared_processors
+        + [
+            structlog.stdlib.ProcessorFormatter.wrap_for_formatter,
+        ],
         context_class=dict,
         logger_factory=structlog.stdlib.LoggerFactory(),
         wrapper_class=structlog.stdlib.BoundLogger,

server/vbv_lernwelt/core/middleware/security.py

@@ -17,7 +17,7 @@ class GetIpBehindReverseProxyMiddleWare:
 
     def __call__(self, request):
         client_ip, _is_routable = get_client_ip(request)
-        request.META['REMOTE_ADDR'] = client_ip
+        request.META["REMOTE_ADDR"] = client_ip
 
         response = self.get_response(request)
 
@@ -100,9 +100,8 @@ class SecurityRequestResponseLoggingMiddleware:
 
         self._raised_exception = True
 
-        self.bind_user_id(request),
         logger.exception(
             "request_failed",
-            code=500,
-            request=self.format_request(request),
+            label="security",
+            response_status_code=500,
         )

server/vbv_lernwelt/core/utils.py

@@ -20,7 +20,7 @@ def structlog_inject_context_dict(test, level, event_dict):
     """
     Add the structlog context dict to log events generated by the stdlib logging library.
     """
-    context_class = structlog.get_config().get('context_class')
+    context_class = structlog.get_config().get("context_class")
 
     if context_class:
         for key, value in context_class().items():

server/vbv_lernwelt/simpletodo/serializers.py

@@ -31,5 +31,11 @@ class SimpleTaskSerializer(ModelSerializer):
 
         validated_data["list"] = simple_list
 
-        logger.debug("Creating task", title=validated_data.get('title'), list_title=list_title)
+        logger.debug(
+            "Creating task",
+            label="simpletodo",
+            dt={"s1": 3, "s2": 4},
+            title=validated_data.get("title"),
+            list_title=list_title,
+        )
         return super().create(validated_data)