fix log

app/core/logger.py

@@ -28,8 +28,8 @@ class Logger:
         self.log_level = log_level.upper()
         self.environment = os.getenv("ENVIRONMENT", "PROD").upper()
 
-        # Skip file logging in Hugging Face environment
-        if self.environment == "HUGGINGFACE":
+        # Skip file logging in production/Hugging Face environment
+        if self.environment in ["PROD", "HUGGINGFACE"]:
             self.log_file_path = None
         else:
             self.log_file_path = os.getenv("LOG_FILE_PATH", self._get_default_log_file_path())
@@ -39,14 +39,9 @@ class Logger:
         if self.environment == "DEV":
             # For development, use local logs directory
             log_dir = Path("logs")
-        else:
-            # For production, use logs directory within app directory
-            log_dir = Path("/opt/MailPilot/logs")
-        
-        # Create logs directory if it doesn't exist
-        log_dir.mkdir(parents=True, exist_ok=True)
-        
-        return str(log_dir / "app.log")
+            log_dir.mkdir(parents=True, exist_ok=True)
+            return str(log_dir / "app.log")
+        return None
 
     def _rename_event_key(self, _, __, event_dict: EventDict) -> EventDict:
         """
@@ -112,12 +107,7 @@ class Logger:
         )
 
     def _configure_logging(self, processors: list[Processor]) -> logging.Logger:
-        """
-        Configures logging with the specified processors based on the environment.
-
-        Returns:
-            logging.Logger: The configured root logger.
-        """
+        """Configures logging with the specified processors."""
         formatter = structlog.stdlib.ProcessorFormatter(
             foreign_pre_chain=processors,
             processors=[
@@ -136,8 +126,8 @@ class Logger:
         stream_handler.setFormatter(formatter)
         root_logger.addHandler(stream_handler)
 
-        # Only add file logging if not in Hugging Face environment and log_file_path is set
-        if self.environment != "HUGGINGFACE" and self.log_file_path:
+        # Only add file logging if in development and log_file_path is set
+        if self.environment == "DEV" and self.log_file_path:
             try:
                 file_handler = TimedRotatingFileHandler(
                     filename=self.log_file_path,
@@ -149,7 +139,7 @@ class Logger:
                 file_handler.setFormatter(formatter)
                 root_logger.addHandler(file_handler)
             except PermissionError:
-                # If file logging fails, just continue with console logging
+                # If file logging fails, continue with console logging only
                 pass
 
         root_logger.setLevel(self.log_level)