From d53ea88598c087a218b0b09e20ba8196d14886af Mon Sep 17 00:00:00 2001 From: Zhenting Huang <3061613175@qq.com> Date: Sat, 21 Feb 2026 16:50:40 +0800 Subject: [PATCH] fix(bedrock): prevent double UUID in create_file S3 key (#21650) In create_file for Bedrock, get_complete_file_url is called twice: once in the sync handler (generating UUID-1 for api_base) and once inside transform_create_file_request (generating UUID-2 for the actual S3 upload). The Bedrock provider correctly writes UUID-2 into litellm_params["upload_url"], but the sync handler unconditionally overwrites it with api_base (UUID-1). This causes the returned file_id to point to a non-existent S3 key. Fix: only set upload_url to api_base when transform_create_file_request has not already set it, preserving the Bedrock provider's value. Closes #21546 --- litellm/llms/custom_httpx/llm_http_handler.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/litellm/llms/custom_httpx/llm_http_handler.py b/litellm/llms/custom_httpx/llm_http_handler.py index 0a5364bfcfe..5087f2e0078 100644 --- a/litellm/llms/custom_httpx/llm_http_handler.py +++ b/litellm/llms/custom_httpx/llm_http_handler.py @@ -3014,8 +3014,11 @@ class BaseLLMHTTPHandler: raise ValueError(f"Unsupported transformed_request type: {type(transformed_request)}") # Store the upload URL in litellm_params for the transformation method + # Honour the URL already set by transform_create_file_request (e.g. Bedrock pre-signed S3 uploads), + # fall back to api_base for providers that do not set it. litellm_params_with_url = dict(litellm_params) - litellm_params_with_url["upload_url"] = api_base + if "upload_url" not in litellm_params: + litellm_params_with_url["upload_url"] = api_base return provider_config.transform_create_file_response( model=None,