diff --git a/tests/proxy_unit_tests/test_proxy_server.py b/tests/proxy_unit_tests/test_proxy_server.py index b55259afee..d7e338d657 100644 --- a/tests/proxy_unit_tests/test_proxy_server.py +++ b/tests/proxy_unit_tests/test_proxy_server.py @@ -2497,9 +2497,6 @@ async def test_get_config_callbacks_with_all_types(client_no_auth): with patch.object( proxy_config, "get_config", new=AsyncMock(return_value=mock_config_data) - ), patch( - "litellm.proxy.common_utils.callback_utils.decrypt_value_helper", - side_effect=lambda value, key=None: value ): response = client_no_auth.get("/get/config/callbacks") @@ -2549,7 +2546,7 @@ async def test_get_config_callbacks_with_all_types(client_no_auth): async def test_get_config_callbacks_environment_variables(client_no_auth): """ Test that /get/config/callbacks correctly includes environment variables - for each callback type with proper decryption. + for each callback type. Values are returned as-is from the config (no decryption). """ from litellm.proxy.proxy_server import ProxyConfig @@ -2561,8 +2558,8 @@ async def test_get_config_callbacks_environment_variables(client_no_auth): "callbacks": ["otel"] }, "environment_variables": { - "LANGFUSE_PUBLIC_KEY": "encrypted-public-key", - "LANGFUSE_SECRET_KEY": "encrypted-secret-key", + "LANGFUSE_PUBLIC_KEY": "test-public-key", + "LANGFUSE_SECRET_KEY": "test-secret-key", "LANGFUSE_HOST": "https://cloud.langfuse.com", "OTEL_EXPORTER": "otlp", "OTEL_ENDPOINT": "http://localhost:4317", @@ -2571,19 +2568,10 @@ async def test_get_config_callbacks_environment_variables(client_no_auth): "general_settings": {} } - # Mock decrypt to prepend "decrypted-" to values - def mock_decrypt(value, key=None): - if value and isinstance(value, str) and "encrypted" in value: - return f"decrypted-{value}" - return value - proxy_config = getattr(litellm.proxy.proxy_server, "proxy_config") with patch.object( proxy_config, "get_config", new=AsyncMock(return_value=mock_config_data) - ), patch( - "litellm.proxy.common_utils.callback_utils.decrypt_value_helper", - side_effect=mock_decrypt ): response = client_no_auth.get("/get/config/callbacks") @@ -2600,12 +2588,12 @@ async def test_get_config_callbacks_environment_variables(client_no_auth): assert langfuse_callback["type"] == "success" assert "variables" in langfuse_callback - # Verify langfuse env vars are present and decrypted + # Verify langfuse env vars are present (values returned as-is, no decryption) langfuse_vars = langfuse_callback["variables"] assert "LANGFUSE_PUBLIC_KEY" in langfuse_vars - assert langfuse_vars["LANGFUSE_PUBLIC_KEY"] == "decrypted-encrypted-public-key" + assert langfuse_vars["LANGFUSE_PUBLIC_KEY"] == "test-public-key" assert "LANGFUSE_SECRET_KEY" in langfuse_vars - assert langfuse_vars["LANGFUSE_SECRET_KEY"] == "decrypted-encrypted-secret-key" + assert langfuse_vars["LANGFUSE_SECRET_KEY"] == "test-secret-key" assert "LANGFUSE_HOST" in langfuse_vars assert langfuse_vars["LANGFUSE_HOST"] == "https://cloud.langfuse.com"