Skip to content

fix(bedrock): retrieve managed file content via presigned S3 GET (#26335) #31433

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kingdoooo wants to merge 13 commits into from
Closed

fix(bedrock): retrieve managed file content via presigned S3 GET (#26335) #31433

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
762881b
docs: design spec for bedrock managed file content retrieval (#26335)
kingdoooo Jun 22, 2026
3b211e0
docs: commit to full bedrock batch e2e proof of fix (#26335)
kingdoooo Jun 22, 2026
66e9076
docs: implementation plan for bedrock file content retrieval (#26335)
kingdoooo Jun 22, 2026
5b4a602
docs: harden bedrock presign (partition endpoint, sigv4, expiry) afte…
kingdoooo Jun 22, 2026
f9c34bf
docs: revise bedrock file content design+plan per codex findings (#26…
kingdoooo Jun 24, 2026
a2187fb
refactor(bedrock): add trusted-snapshot S3 bucket/region resolution t…
kingdoooo Jun 24, 2026
e9e9777
feat(bedrock): retrieve managed file content via presigned S3 GET (#2…
kingdoooo Jun 24, 2026
3a32973
test(bedrock): cover file content retrieval through generic handler
kingdoooo Jun 24, 2026
2072de0
refactor(bedrock): delete dead BedrockFilesHandler and unreachable fi…
kingdoooo Jun 24, 2026
ca10f2f
style(bedrock): black-format test_bedrock_files_handler line wrapping
kingdoooo Jun 24, 2026
49c2d07
docs: correct presigned-URL log-exposure surface per final review (#2…
kingdoooo Jun 24, 2026
09932d7
fix(bedrock): clarify S3 bucket resolution error for file content ret…
kingdoooo Jun 24, 2026
333622c
chore: drop internal SDD scaffolding from PR branch
kingdoooo Jun 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 0 additions & 11 deletions litellm/files/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@
from litellm.litellm_core_utils.litellm_logging import Logging as LiteLLMLoggingObj
from litellm.llms.azure.common_utils import get_azure_credentials
from litellm.llms.azure.files.handler import AzureOpenAIFilesAPI
from litellm.llms.bedrock.files.handler import BedrockFilesHandler
from litellm.llms.custom_httpx.http_handler import AsyncHTTPHandler, HTTPHandler
from litellm.llms.custom_httpx.llm_http_handler import BaseLLMHTTPHandler
from litellm.llms.openai.common_utils import get_openai_credentials
Expand Down Expand Up @@ -82,7 +81,6 @@ def _should_sdk_support_streaming(
openai_files_instance = OpenAIFilesAPI()
azure_files_instance = AzureOpenAIFilesAPI()
vertex_ai_files_instance = VertexAIFilesHandler()
bedrock_files_instance = BedrockFilesHandler()
#################################################


Expand Down Expand Up @@ -1018,15 +1016,6 @@ def file_content(
max_retries=optional_params.max_retries,
litellm_params=litellm_params_dict,
)
elif custom_llm_provider == "bedrock":
response = bedrock_files_instance.file_content(
_is_async=_is_async,
file_content_request=_file_content_request,
api_base=optional_params.api_base,
optional_params=litellm_params_dict,
timeout=timeout,
max_retries=optional_params.max_retries,
)
else:
raise litellm.exceptions.BadRequestError(
message="LiteLLM doesn't support {} for 'file_content'. Supported providers are 'openai', 'azure', 'vertex_ai', 'bedrock', 'manus', 'anthropic'.".format(
Expand Down
242 changes: 0 additions & 242 deletions litellm/llms/bedrock/files/handler.py
View file
Open in desktop

This file was deleted.

Loading