fix logging issue

src/application/error_handling/structured_logger.py

@@ -124,21 +124,24 @@ class StructuredLogger:
         """Log debug message."""
         if self.logger.isEnabledFor(logging.DEBUG):
             log_data = self._get_log_data(message, LogLevel.DEBUG.value, context, extra)
-            self.logger.debug(message, extra=log_data)
+            # Use 'structured_data' to avoid conflicts with LogRecord attributes
+            self.logger.debug(message, extra={'structured_data': log_data})
 
     def info(self, message: str, context: Optional[LogContext] = None,
             extra: Optional[Dict[str, Any]] = None) -> None:
         """Log info message."""
         if self.logger.isEnabledFor(logging.INFO):
             log_data = self._get_log_data(message, LogLevel.INFO.value, context, extra)
-            self.logger.info(message, extra=log_data)
+            # Use 'structured_data' to avoid conflicts with LogRecord attributes
+            self.logger.info(message, extra={'structured_data': log_data})
 
     def warning(self, message: str, context: Optional[LogContext] = None,
                extra: Optional[Dict[str, Any]] = None) -> None:
         """Log warning message."""
         if self.logger.isEnabledFor(logging.WARNING):
             log_data = self._get_log_data(message, LogLevel.WARNING.value, context, extra)
-            self.logger.warning(message, extra=log_data)
+            # Use 'structured_data' to avoid conflicts with LogRecord attributes
+            self.logger.warning(message, extra={'structured_data': log_data})
 
     def error(self, message: str, context: Optional[LogContext] = None,
              extra: Optional[Dict[str, Any]] = None,
@@ -146,7 +149,8 @@ class StructuredLogger:
         """Log error message."""
         if self.logger.isEnabledFor(logging.ERROR):
             log_data = self._get_log_data(message, LogLevel.ERROR.value, context, extra, exception)
-            self.logger.error(message, extra=log_data)
+            # Use 'structured_data' to avoid conflicts with LogRecord attributes
+            self.logger.error(message, extra={'structured_data': log_data})
 
     def critical(self, message: str, context: Optional[LogContext] = None,
                 extra: Optional[Dict[str, Any]] = None,
@@ -154,7 +158,8 @@ class StructuredLogger:
         """Log critical message."""
         if self.logger.isEnabledFor(logging.CRITICAL):
             log_data = self._get_log_data(message, LogLevel.CRITICAL.value, context, extra, exception)
-            self.logger.critical(message, extra=log_data)
+            # Use 'structured_data' to avoid conflicts with LogRecord attributes
+            self.logger.critical(message, extra={'structured_data': log_data})
 
     def log_operation_start(self, operation: str, context: Optional[LogContext] = None,
                           extra: Optional[Dict[str, Any]] = None) -> str:
@@ -256,7 +261,7 @@ class JsonFormatter(logging.Formatter):
         """Format log record as JSON."""
         try:
             # Get structured data from extra
-            log_data = getattr(record, 'extra', {})
+            log_data = getattr(record, 'structured_data', {})
 
             # Ensure basic fields are present
             if 'timestamp' not in log_data:
@@ -300,16 +305,16 @@ class ContextFormatter(logging.Formatter):
             # Get correlation ID from context or record
             correlation_id = correlation_id_context.get()
             if not correlation_id:
-                extra_data = getattr(record, 'extra', {})
-                correlation_id = extra_data.get('correlation_id', 'unknown')
+                structured_data = getattr(record, 'structured_data', {})
+                correlation_id = structured_data.get('correlation_id', 'unknown')
 
             # Add correlation ID to record
             record.correlation_id = correlation_id
 
             # Add context information if available
-            extra_data = getattr(record, 'extra', {})
-            if 'operation' in extra_data:
-                record.message = f"[{extra_data['operation']}] {record.getMessage()}"
+            structured_data = getattr(record, 'structured_data', {})
+            if 'operation' in structured_data:
+                record.message = f"[{structured_data['operation']}] {record.getMessage()}"
 
             return super().format(record)