Skip to content

[Fix] Claude Code (/messages) - Litellm fix claude code Bedrock Invoke usage, request signing - #19111

Merged
ishaan-jaff merged 5 commits into
mainfrom
litellm_fix_claude_code_invoke_usage
Jan 14, 2026
Merged

[Fix] Claude Code (/messages) - Litellm fix claude code Bedrock Invoke usage, request signing #19111
ishaan-jaff merged 5 commits into
mainfrom
litellm_fix_claude_code_invoke_usage

Conversation

@ishaan-jaff

@ishaan-jaff ishaan-jaff commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

[Fix] Claude Code (/messages) - Litellm fix claude code Bedrock Invoke usage, request signing

Fixes Bedrock invoke SigV4 signing to match the converse path by filtering headers before signing and keeping forwarded headers unsigned. This prevents signature mismatches when forward_client_headers_to_llm_api is enabled and ALB mutates x-forwarded-* headers.

Screenshot 2026-01-14 at 2 22 46 PM

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🐛 Bug Fix
✅ Test

Changes

@vercel

vercel Bot commented Jan 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
litellm Ready Ready Preview, Comment Jan 14, 2026 10:23pm

@ishaan-jaff
ishaan-jaff merged commit 06ded87 into main Jan 14, 2026
47 of 60 checks passed
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…e usage, request signing (BerriAI#19111)

* test_should_not_fail_with_forwarded_headers_bedrock_invoke_messages

* use common get_request_headers for BaseAWS

* fix get_request_headers

* test_should_not_fail_with_forwarded_headers_bedrock_invoke_messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant