From 99f007f51d9961a3818b43ecb2bb74e6d69a2418 Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Mon, 20 Apr 2026 16:59:54 -0700 Subject: [PATCH] refactor: consolidate redirect_uri scheme check into shared handler --- .../management_endpoints/mcp_management_endpoints.py | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/litellm/proxy/management_endpoints/mcp_management_endpoints.py b/litellm/proxy/management_endpoints/mcp_management_endpoints.py index 9cb5ec95ab9..8e54af0f96a 100644 --- a/litellm/proxy/management_endpoints/mcp_management_endpoints.py +++ b/litellm/proxy/management_endpoints/mcp_management_endpoints.py @@ -19,7 +19,6 @@ import functools import importlib import json import os -from urllib.parse import urlparse from dataclasses import dataclass from datetime import datetime, timedelta, timezone from typing import Any, Dict, Iterable, List, Literal, Optional @@ -1379,15 +1378,6 @@ if MCP_AVAILABLE: response_type: Optional[str] = None, scope: Optional[str] = None, ): - parsed_redirect = urlparse(redirect_uri) - if parsed_redirect.scheme not in ("http", "https"): - raise HTTPException( - status_code=status.HTTP_400_BAD_REQUEST, - detail={ - "error": "invalid_redirect_uri", - "message": "redirect_uri must use http or https scheme", - }, - ) mcp_server = _get_cached_temporary_mcp_server_or_404(server_id, request=request) # Use the server's stored client_id when the caller doesn't supply one resolved_client_id = mcp_server.client_id or client_id or ""