Adjust Redis connection logic for SSL-first approach, add Hugging Face provider debugging, and update requirements

core/providers/huggingface.py

@@ -19,6 +19,10 @@ class HuggingFaceProvider(LLMProvider):
     def __init__(self, model_name: str, timeout: int = 30, max_retries: int = 3):
         super().__init__(model_name, timeout, max_retries)
         
+        logger.info(f"Initializing HuggingFaceProvider with:")
+        logger.info(f"  HF_API_URL: {config.hf_api_url}")
+        logger.info(f"  HF_TOKEN SET: {bool(config.hf_token)}")
+        
         if not HUGGINGFACE_SDK_AVAILABLE:
             raise ImportError("Hugging Face provider requires 'openai' package")
             
@@ -26,10 +30,16 @@ class HuggingFaceProvider(LLMProvider):
             raise ValueError("HF_TOKEN not set - required for Hugging Face provider")
             
         # Make sure NO proxies parameter is included
-        self.client = OpenAI(
-            base_url=config.hf_api_url,
-            api_key=config.hf_token
-        )
+        try:
+            self.client = OpenAI(
+                base_url=config.hf_api_url,
+                api_key=config.hf_token
+            )
+            logger.info("HuggingFaceProvider initialized successfully")
+        except Exception as e:
+            logger.error(f"Failed to initialize HuggingFaceProvider: {e}")
+            logger.error(f"Error type: {type(e)}")
+            raise
     
     def generate(self, prompt: str, conversation_history: List[Dict]) -> Optional[str]:
         """Generate a response synchronously"""

core/redis_client.py

@@ -36,13 +36,37 @@ class RedisClient:
             logger.info("Redis not configured, skipping connection")
             return None
 
+        # If SSL is explicitly disabled, try non-SSL only
+        if config.redis_disable_ssl:
+            try:
+                host, port = self._parse_host_port(config.redis_host, config.redis_port)
+                logger.info(f"Connecting to Redis at {host}:{port} (SSL disabled)")
+                
+                self._redis_client = redis.Redis(
+                    host=host,
+                    port=port,
+                    username=config.redis_username or None,
+                    password=config.redis_password or None,
+                    decode_responses=True,
+                    socket_connect_timeout=5,
+                    socket_timeout=5,
+                    retry_on_timeout=True,
+                    health_check_interval=30
+                )
+                
+                self._redis_client.ping()
+                logger.info("Successfully connected to Redis without SSL")
+                return
+            except Exception as e:
+                logger.error(f"Non-SSL connection failed: {e}")
+                self._redis_client = None
+                return
+
+        # Try SSL connection first (default for Redis Cloud)
         try:
-            # Parse host and port if port is included in host
             host, port = self._parse_host_port(config.redis_host, config.redis_port)
+            logger.info(f"Connecting to Redis at {host}:{port} with SSL")
             
-            logger.info(f"Connecting to Redis at {host}:{port}")
-            
-            # Try connection WITHOUT SSL first for Redis Cloud (most common setup)
             self._redis_client = redis.Redis(
                 host=host,
                 port=port,
@@ -51,18 +75,19 @@ class RedisClient:
                 decode_responses=True,
                 socket_connect_timeout=5,
                 socket_timeout=5,
+                ssl=True,
+                ssl_cert_reqs=None,  # Skip cert verification for Redis Cloud
                 retry_on_timeout=True,
                 health_check_interval=30
             )
             
-            # Test connection
             self._redis_client.ping()
-            logger.info("Successfully connected to Redis without SSL")
+            logger.info("Successfully connected to Redis with SSL")
+            return
             
         except Exception as e:
-            logger.warning(f"Non-SSL connection failed: {e}, trying with SSL")
+            logger.warning(f"SSL connection failed: {e}, trying without SSL")
             try:
-                # Fallback to SSL connection
                 host, port = self._parse_host_port(config.redis_host, config.redis_port)
                 
                 self._redis_client = redis.Redis(
@@ -73,15 +98,13 @@ class RedisClient:
                     decode_responses=True,
                     socket_connect_timeout=5,
                     socket_timeout=5,
-                    ssl=True,
-                    ssl_cert_reqs=None,  # Skip cert verification for Redis Cloud
                     retry_on_timeout=True,
                     health_check_interval=30
                 )
                 
-                # Test connection
                 self._redis_client.ping()
-                logger.info("Successfully connected to Redis with SSL")
+                logger.info("Successfully connected to Redis without SSL")
+                return
                 
             except Exception as e2:
                 logger.error(f"Could not connect to Redis: {e2}")

requirements.txt

@@ -3,7 +3,7 @@ fastapi==0.95.0
 uvicorn==0.21.1
 redis==5.0.3
 python-dotenv==1.0.0
-openai==1.35.6
+openai>=1.35.6
 tavily-python>=0.1.0,<1.0.0
 requests==2.31.0
 docker==6.1.3