Skip to content

docs(proxy): require model_info.id and drop the status code from the PTU page - #968

Merged
yucheng-berri merged 2 commits into
mainfrom
litellm_ptu_docs_drop_status_code
Aug 21, 2026
Merged

docs(proxy): require model_info.id and drop the status code from the PTU page#968
yucheng-berri merged 2 commits into
mainfrom
litellm_ptu_docs_drop_status_code

Conversation

@yucheng-berri

@yucheng-berri yucheng-berri commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The PTU page says a rejected rate answers 400. Some invalid PTU configurations answer 422 instead, so naming one code sends readers looking for a response they may not get

Removes the code and keeps the part that is reliably true, that the rejection names the field it refused. No behaviour change is implied here; this is a docs correction only, and whether the endpoints should be normalised onto a single code is a separate question


Open in Devin Review

@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview Aug 21, 2026 7:05pm

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Some invalid PTU configurations answer 422 rather than 400, so naming a
single code on the page is misleading. The behaviour that matters is that
the rejection names the offending field.
BerriAI/litellm#37794 refuses a config-declared PTU deployment that does
not carry an id, so the page can no longer present pinning one as advice.
Adds the upgrade note, since setting a fresh name rather than the id the
deployment already uses strands the charges already written
@yucheng-berri yucheng-berri changed the title docs(proxy): drop the status code from the PTU rejection note docs(proxy): require model_info.id and drop the status code from the PTU page Aug 21, 2026
@yucheng-berri
yucheng-berri force-pushed the litellm_ptu_docs_drop_status_code branch from 035118d to 6d76fd9 Compare August 21, 2026 18:56
@yucheng-berri
yucheng-berri merged commit a269d73 into main Aug 21, 2026
4 checks passed
@yucheng-berri
yucheng-berri deleted the litellm_ptu_docs_drop_status_code branch August 21, 2026 19:08
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