reverted custom logging

Author: AjitPadhi-Microsoft

content-gen/.env.sample

@@ -113,13 +113,3 @@ WORKERS=4
 # Feature flags
 AUTH_ENABLED=false
 SANITIZE_ANSWER=false
-
-# =============================================================================
-# Logging Configuration
-# =============================================================================
-# Basic logging level (DEBUG, INFO, WARNING, ERROR)
-AZURE_BASIC_LOGGING_LEVEL=INFO
-# Logging level for Azure SDK and third-party packages (DEBUG, INFO, WARNING, ERROR)
-AZURE_PACKAGE_LOGGING_LEVEL=WARNING
-# Comma-separated list of Python logger names to apply package logging level to
-AZURE_LOGGING_PACKAGES=

content-gen/infra/main.bicep

@@ -868,10 +868,6 @@ module containerInstance 'modules/container-instance.bicep' = {
       { name: 'AZURE_AI_PROJECT_ENDPOINT', value: aiFoundryAiProjectEndpoint }
       { name: 'AZURE_AI_MODEL_DEPLOYMENT_NAME', value: gptModelName }
       { name: 'AZURE_AI_IMAGE_MODEL_DEPLOYMENT', value: imageModelConfig[imageModelChoice].name }
-      // Logging Settings
-      { name: 'AZURE_BASIC_LOGGING_LEVEL', value: 'INFO' }
-      { name: 'AZURE_PACKAGE_LOGGING_LEVEL', value: 'WARNING' }
-      { name: 'AZURE_LOGGING_PACKAGES', value: '' }
       // Application Insights
       { name: 'APPLICATIONINSIGHTS_CONNECTION_STRING', value: enableMonitoring ? applicationInsights!.outputs.connectionString : '' }
     ]

content-gen/infra/main.json

@@ -6,7 +6,7 @@
     "_generator": {
       "name": "bicep",
       "version": "0.41.2.15936",
-      "templateHash": "1650354659075455837"
+      "templateHash": "11360056711318509175"
     },
     "name": "Intelligent Content Generation Accelerator",
     "description": "Solution Accelerator for multimodal marketing content generation using Microsoft Agent Framework.\n"
@@ -33358,18 +33358,6 @@
                 "name": "AZURE_AI_IMAGE_MODEL_DEPLOYMENT",
                 "value": "[variables('imageModelConfig')[parameters('imageModelChoice')].name]"
               },
-              {
-                "name": "AZURE_BASIC_LOGGING_LEVEL",
-                "value": "INFO"
-              },
-              {
-                "name": "AZURE_PACKAGE_LOGGING_LEVEL",
-                "value": "WARNING"
-              },
-              {
-                "name": "AZURE_LOGGING_PACKAGES",
-                "value": ""
-              },
               {
                 "name": "APPLICATIONINSIGHTS_CONNECTION_STRING",
                 "value": "[if(parameters('enableMonitoring'), reference('applicationInsights').outputs.connectionString.value, '')]"

content-gen/src/backend/app.py

@@ -35,22 +35,11 @@
 
 _active_regenerations: Dict[str, Dict[str, Any]] = {}
 
-logging_settings = app_settings.logging
-# Configure logging based on environment variables
+# Configure logging
 logging.basicConfig(
-    level=logging_settings.get_basic_log_level(),
-    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
-    force=True
+    level=logging.INFO,
+    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s"
 )
-azure_log_level = logging_settings.get_package_log_level()
-for logger_name in logging_settings.logging_packages or []:
-    logging.getLogger(logger_name).setLevel(azure_log_level)
-logging.info(
-    f"Logging configured - Basic: {logging_settings.basic_logging_level}, "
-    f"Azure packages: {logging_settings.package_logging_level}, "
-    f"Packages: {logging_settings.logging_packages}"
-)
-
 logger = logging.getLogger(__name__)
 
 # Create Quart app

content-gen/src/backend/settings.py

@@ -6,11 +6,10 @@
 and compliance validation.
 """
 
-import logging
 import os
-from typing import List, Literal, Optional
+from typing import List, Optional
 
-from pydantic import BaseModel, Field, field_validator, model_validator
+from pydantic import BaseModel, Field, model_validator
 from pydantic_settings import BaseSettings, SettingsConfigDict
 from typing_extensions import Self
 
@@ -26,32 +25,6 @@ def parse_comma_separated(value: str) -> List[str]:
     return []
 
 
-class _LoggingSettings(BaseSettings):
-    """Logging configuration settings."""
-    model_config = SettingsConfigDict(
-        env_prefix="AZURE_", env_file=DOTENV_PATH, extra="ignore", env_ignore_empty=True
-    )
-
-    basic_logging_level: Literal["DEBUG", "INFO", "WARNING", "ERROR"] = "INFO"
-    package_logging_level: Literal["DEBUG", "INFO", "WARNING", "ERROR"] = "WARNING"
-    logging_packages: Optional[List[str]] = []
-
-    @field_validator("logging_packages", mode="before")
-    @classmethod
-    def split_logging_packages(cls, packages) -> Optional[List[str]]:
-        if isinstance(packages, str) and len(packages.strip()) > 0:
-            return [pkg.strip() for pkg in packages.split(",") if pkg.strip()]
-        return None
-
-    def get_basic_log_level(self) -> int:
-        """Convert string log level to logging constant."""
-        return getattr(logging, self.basic_logging_level.upper())
-
-    def get_package_log_level(self) -> int:
-        """Convert string package log level to logging constant."""
-        return getattr(logging, self.package_logging_level.upper())
-
-
 class _UiSettings(BaseSettings):
     """UI configuration settings."""
     model_config = SettingsConfigDict(
@@ -463,7 +436,6 @@ class _AppSettings(BaseModel):
     azure_openai: _AzureOpenAISettings = _AzureOpenAISettings()
     ai_foundry: _AIFoundrySettings = _AIFoundrySettings()
     brand_guidelines: _BrandGuidelinesSettings = _BrandGuidelinesSettings()
-    logging: _LoggingSettings = _LoggingSettings()
     ui: Optional[_UiSettings] = _UiSettings()
 
     # Constructed properties