Skip to content

docs(virtual_keys): document allowed_routes in default_key_generate_params - #1477

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
litellm_default_key_generate_params_allowed_routes
Open

devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
litellm_default_key_generate_params_allowed_routes

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Documents the new allowed_routes support in litellm_settings.default_key_generate_params (BerriAI/litellm branch litellm_default_key_generate_params_allowed_routes, Linear LIT-7788). Adds the field to the yaml example in docs/proxy/virtual_keys.md and a sentence explaining that it fills unset requests, that a non-empty value is an allowlist (so llm_api_routes must be included alongside /key/spend/report), and that a key_type preset on the request takes precedence

Link to Devin session: https://app.devin.ai/sessions/efc6675e3f474edeabfb8e84de0855e4
Open in Devin Desktop: https://app.devin.ai/desktop/session/efc6675e3f474edeabfb8e84de0855e4?variant=devin


Note

Low Risk
Documentation-only; no runtime or config parsing behavior changes in this diff.

Overview
Updates docs/proxy/virtual_keys.md so litellm_settings.default_key_generate_params documents allowed_routes: defaults apply when the field is missing or null (like other params, with the same budget_duration exception noted elsewhere). The new text explains using ["llm_api_routes", "/key/spend/report"] so new keys can report spend, that a non-empty value is an allowlist (so route groups such as llm_api_routes must be included), and that an explicit key_type on the request wins over the default.

The sample default_key_generate_params YAML now includes allowed_routes. A stray blank line under the Teams API reference heading is removed.

Reviewed by Cursor Bugbot for commit 5f67fe3. Bugbot is set up for automated code reviews on this repo. Configure here.

…arams

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@vercel

vercel Bot commented Sep 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
litellm Ready Ready Preview Sep 15, 2026 1:19pm UTC

Request Review

@mateo-berri

Copy link
Copy Markdown
Contributor

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 5f67fe3. Configure here.

This branch was successfully deployed

1 active deployment
Preview 5f67fe31 Deployed Sep 15, 2026 by vercel[bot]
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