Skip to content

Conversation

stevenmanton
Copy link
Contributor

@stevenmanton stevenmanton commented Aug 24, 2025

Title

Bug fix: AWS assume_role takes token

Relevant issues

Fixes #13918

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
  • I have added a screenshot of my new test passing locally
  • 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
image

Note: All but one test passes. It looks like the failing test is an issue with a missing module, which is unrelated to my change.

Type

🐛 Bug Fix

Changes

I added support for passing in the AWS session token. I confirmed that this worked as expected in my environment and that I could successfully daisychain assumed-role sessions.

Copy link

vercel bot commented Aug 24, 2025

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

Project Deployment Preview Comments Updated (UTC)
litellm Ready Ready Preview Comment Aug 24, 2025 5:38am

@CLAassistant
Copy link

CLAassistant commented Aug 24, 2025

CLA assistant check
All committers have signed the CLA.

@stevenmanton stevenmanton force-pushed the fix-aws-assume-role-with-token branch from f5bdf3b to d955784 Compare August 24, 2025 05:37
@krrishdholakia krrishdholakia merged commit 0d285c8 into BerriAI:main Aug 25, 2025
6 checks passed
@stevenmanton stevenmanton deleted the fix-aws-assume-role-with-token branch August 29, 2025 03:11
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.

[Bug]: AWS assume role does not include the session token
3 participants