From 76a4415ecde8dc993c8262d9d79b14dadce6b572 Mon Sep 17 00:00:00 2001 From: Ishaan Jaffer Date: Wed, 11 Mar 2026 20:54:44 -0700 Subject: [PATCH] fix(mcp): remove user-provided values from debug log to fix log-injection lint --- .../_experimental/mcp_server/rest_endpoints.py | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/litellm/proxy/_experimental/mcp_server/rest_endpoints.py b/litellm/proxy/_experimental/mcp_server/rest_endpoints.py index 5e4d88355a7..0c62f5d5dac 100644 --- a/litellm/proxy/_experimental/mcp_server/rest_endpoints.py +++ b/litellm/proxy/_experimental/mcp_server/rest_endpoints.py @@ -100,12 +100,7 @@ if MCP_AVAILABLE: if cred and cred.get("access_token"): return {"Authorization": f"Bearer {cred['access_token']}"} except Exception: - verbose_logger.debug( - "Failed to fetch OAuth credential for user %r / server %r", - str(user_id).replace("\n", "\\n").replace("\r", "\\r"), - str(server_id).replace("\n", "\\n").replace("\r", "\\r"), - exc_info=True, - ) + verbose_logger.debug("Failed to fetch OAuth credential", exc_info=True) return None async def _get_bulk_user_oauth_headers( @@ -137,11 +132,7 @@ if MCP_AVAILABLE: if c.get("access_token") and c.get("server_id") } except Exception: - verbose_logger.debug( - "Failed to bulk-fetch OAuth credentials for user %r", - str(user_id).replace("\n", "\\n").replace("\r", "\\r"), - exc_info=True, - ) + verbose_logger.debug("Failed to bulk-fetch OAuth credentials", exc_info=True) return {} def _create_tool_response_objects(tools, server_mcp_info):