Fix Redis configuration timing issue

- Add reload_redis_config() function to reload REDIS_CFG from environment variables
- Call reload_redis_config() after setting environment variables in CLI
- This ensures REDIS_CFG uses the CLI-provided values instead of initial defaults
- Fixes issue where Redis Cloud URIs were ignored and localhost was used instead

Author: mortensi

src/common/config.py

@@ -8,18 +8,28 @@
 MCP_HOST = os.getenv('MCP_HOST', '127.0.0.1')
 MCP_PORT = os.getenv('MCP_PORT', 8000)
 
-REDIS_CFG = {"host": os.getenv('REDIS_HOST', '127.0.0.1'),
-             "port": int(os.getenv('REDIS_PORT',6379)),
-             "username": os.getenv('REDIS_USERNAME', None),
-             "password": os.getenv('REDIS_PWD',''),
-             "ssl": os.getenv('REDIS_SSL', False) in ('true', '1', 't'),
-             "ssl_ca_path": os.getenv('REDIS_SSL_CA_PATH', None),
-             "ssl_keyfile": os.getenv('REDIS_SSL_KEYFILE', None),
-             "ssl_certfile": os.getenv('REDIS_SSL_CERTFILE', None),
-             "ssl_cert_reqs": os.getenv('REDIS_SSL_CERT_REQS', 'required'),
-             "ssl_ca_certs": os.getenv('REDIS_SSL_CA_CERTS', None),
-             "cluster_mode": os.getenv('REDIS_CLUSTER_MODE', False) in ('true', '1', 't'),
-             "db": int(os.getenv('REDIS_DB', 0))}
+def _load_redis_config():
+    """Load Redis configuration from environment variables."""
+    return {"host": os.getenv('REDIS_HOST', '127.0.0.1'),
+            "port": int(os.getenv('REDIS_PORT',6379)),
+            "username": os.getenv('REDIS_USERNAME', None),
+            "password": os.getenv('REDIS_PWD',''),
+            "ssl": os.getenv('REDIS_SSL', False) in ('true', '1', 't'),
+            "ssl_ca_path": os.getenv('REDIS_SSL_CA_PATH', None),
+            "ssl_keyfile": os.getenv('REDIS_SSL_KEYFILE', None),
+            "ssl_certfile": os.getenv('REDIS_SSL_CERTFILE', None),
+            "ssl_cert_reqs": os.getenv('REDIS_SSL_CERT_REQS', 'required'),
+            "ssl_ca_certs": os.getenv('REDIS_SSL_CA_CERTS', None),
+            "cluster_mode": os.getenv('REDIS_CLUSTER_MODE', False) in ('true', '1', 't'),
+            "db": int(os.getenv('REDIS_DB', 0))}
+
+REDIS_CFG = _load_redis_config()
+
+
+def reload_redis_config():
+    """Reload Redis configuration from environment variables."""
+    global REDIS_CFG
+    REDIS_CFG = _load_redis_config()
 
 
 def parse_redis_uri(uri: str) -> dict:
@@ -104,3 +114,4 @@ def set_redis_env_from_config(config: dict):
             if isinstance(value, bool):
                 value = 'true' if value else 'false'
             os.environ[env_var] = str(value)
+            print(f"Setting {env_var} to {value}")

src/main.py

@@ -15,7 +15,7 @@
 import src.tools.set
 import src.tools.stream
 import src.tools.pub_sub
-from src.common.config import MCP_TRANSPORT, parse_redis_uri, set_redis_env_from_config
+from src.common.config import MCP_TRANSPORT, parse_redis_uri, set_redis_env_from_config, reload_redis_config
 
 
 class RedisMCPServer:
@@ -84,6 +84,9 @@ def cli(url, host, port, db, username, password,
 
         set_redis_env_from_config(config)
 
+    # Reload Redis configuration to pick up the new environment variables
+    reload_redis_config()
+
     # Set MCP transport settings
     os.environ['MCP_TRANSPORT'] = mcp_transport
     os.environ['MCP_HOST'] = mcp_host