From 05d95fc15a2711a24b7102ed9b9fc6913ceb163e Mon Sep 17 00:00:00 2001 From: Sameer Kankute Date: Fri, 24 Apr 2026 14:39:46 +0530 Subject: [PATCH] chore(proxy): address greptile feedback on container routing helper Remove redundant model_id guard assignment and drop duplicate provider-aware fallback lookup that repeated earlier router checks. Made-with: Cursor --- litellm/proxy/container_endpoints/handler_factory.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/litellm/proxy/container_endpoints/handler_factory.py b/litellm/proxy/container_endpoints/handler_factory.py index 60c98ca0a46..8b1740f177a 100644 --- a/litellm/proxy/container_endpoints/handler_factory.py +++ b/litellm/proxy/container_endpoints/handler_factory.py @@ -63,11 +63,7 @@ def _get_deployment_credentials_for_model_id(model_id: Any) -> Dict[str, Any]: if deployment and deployment.litellm_params: return deployment.litellm_params.model_dump(exclude_none=True) - # Final fallback: lookup helper that supports provider-aware resolution. - deployment_creds = llm_router.get_deployment_credentials_with_provider( - model_id=str(model_id) - ) - return deployment_creds or {} + return {} def _load_endpoints_config() -> Dict: @@ -233,7 +229,7 @@ async def _process_binary_request( custom_llm_provider=custom_llm_provider, ) - if decoded_model_id and not litellm_params.get("model_id"): + if decoded_model_id: litellm_params["model_id"] = decoded_model_id # Resolve provider credentials for routed deployment, e.g. Azure multi-region api_base.