Revert "reverted custom logging"

This reverts commit bef888a.

Author: AjitPadhi-Microsoft

content-gen/.env.sample

@@ -113,3 +113,13 @@ 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,6 +868,10 @@ 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": "11360056711318509175"
+      "templateHash": "1650354659075455837"
     },
     "name": "Intelligent Content Generation Accelerator",
     "description": "Solution Accelerator for multimodal marketing content generation using Microsoft Agent Framework.\n"
@@ -33358,6 +33358,18 @@
                 "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,11 +35,22 @@
 
 _active_regenerations: Dict[str, Dict[str, Any]] = {}
 
-# Configure logging
+logging_settings = app_settings.logging
+# Configure logging based on environment variables
 logging.basicConfig(
-    level=logging.INFO,
-    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s"
+    level=logging_settings.get_basic_log_level(),
+    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
+    force=True
 )
+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,10 +6,11 @@
 and compliance validation.
 """
 
+import logging
 import os
-from typing import List, Optional
+from typing import List, Literal, Optional
 
-from pydantic import BaseModel, Field, model_validator
+from pydantic import BaseModel, Field, field_validator, model_validator
 from pydantic_settings import BaseSettings, SettingsConfigDict
 from typing_extensions import Self
 
@@ -25,6 +26,32 @@ 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(
@@ -436,6 +463,7 @@ class _AppSettings(BaseModel):
     azure_openai: _AzureOpenAISettings = _AzureOpenAISettings()
     ai_foundry: _AIFoundrySettings = _AIFoundrySettings()
     brand_guidelines: _BrandGuidelinesSettings = _BrandGuidelinesSettings()
+    logging: _LoggingSettings = _LoggingSettings()
     ui: Optional[_UiSettings] = _UiSettings()
 
     # Constructed properties