From db456b33d08022a9591002c1da4b33870d81c3c6 Mon Sep 17 00:00:00 2001 From: Ryan Crabbe Date: Wed, 25 Mar 2026 12:26:05 -0700 Subject: [PATCH] fix: address review feedback on master key check - Downgrade ValueError to critical log (no hard failure on startup) - Change length threshold from 16 to 32 to match warning message - Fix typo: reccomended -> recommended - Remove unnecessary f-string prefix --- litellm/proxy/proxy_server.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/litellm/proxy/proxy_server.py b/litellm/proxy/proxy_server.py index 0b338afefb0..f385b497b59 100644 --- a/litellm/proxy/proxy_server.py +++ b/litellm/proxy/proxy_server.py @@ -3095,13 +3095,13 @@ class ProxyConfig: if master_key is not None and isinstance(master_key, str): if master_key == "sk-1234": - raise ValueError( - f"LITELLM_MASTER_KEY is set to the default master-key granting proxy admin access, please change it to something stronger" - ) - if len(master_key) < 16: + verbose_proxy_logger.critical( + "LITELLM_MASTER_KEY is set to the default master key granting proxy admin access, please change it to something stronger" + ) + if len(master_key) < 32: verbose_proxy_logger.warning( - "A key of at least 32 characters is reccomended for production" - ) + "A key of at least 32 characters is recommended for production" + ) ### USER API KEY CACHE IN-MEMORY TTL ### user_api_key_cache_ttl = general_settings.get(