Skip to content

fix(litellm/caching/caching_handler.py): fix kwargs[litellm_params][p… - #10612

Closed
ghost wants to merge 7 commits into
mainfrom
litellm_dev_05_06_2025_p3
Closed

fix(litellm/caching/caching_handler.py): fix kwargs[litellm_params][p…#10612
ghost wants to merge 7 commits into
mainfrom
litellm_dev_05_06_2025_p3

Conversation

@ghost

@ghost ghost commented May 7, 2025

Copy link
Copy Markdown

…reset_cache_key] not being propagated up the call stack during a cache hit, resulting in the x-litellm-cache-key header not being returned (#8822)

  • fix(litellm/caching/caching_handler.py): fix kwargs[litellm_params][preset_cache_key] not being propogated up the call stack during a cache hit, resulting in the x-litellm-cache-key header not being returned

  • fixes

  • formatting


Title

Relevant issues

Fixes #8570

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

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

…reset_cache_key] not being propagated up the call stack during a cache hit, resulting in the x-litellm-cache-key header not being returned (#8822)

* fix(litellm/caching/caching_handler.py): fix kwargs[litellm_params][preset_cache_key] not being propogated up the call stack during a cache hit, resulting in the x-litellm-cache-key header not being returned

* fixes

* formatting

---------

Co-authored-by: michael rodriguez <mirodrr@amazon.com>
@vercel

vercel Bot commented May 7, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 7, 2025 5:36am

@CLAassistant

CLAassistant commented May 7, 2025

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 3 committers have signed the CLA.

✅ reymondzzzz
❌ Krrish Dholakia
❌ mirodrr2


Krrish Dholakia seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@AleksanderMarek

Copy link
Copy Markdown

Any update on when this is going to be merged?

@nielsbosma

Copy link
Copy Markdown
Contributor

I'm also waiting for this to start working.

@jquinter jquinter closed this Feb 13, 2026
@ishaan-berri
ishaan-berri deleted the litellm_dev_05_06_2025_p3 branch March 26, 2026 21:52
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]: x-litellm-cache-key header not being returned on cache hit

6 participants