Skip to content

docs: clarify which Prisma errors allow_requests_on_db_unavailable covers - #1496

Merged
yassin-berriai merged 1 commit into
mainfrom
litellm_docs_db_unavailable_prisma_query_errors
Sep 17, 2026
Merged

yassin-berriai merged 1 commit into
mainfrom
litellm_docs_db_unavailable_prisma_query_errors

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

The "Gracefully handle DB unavailability" table in docs/proxy/prod.md said every Prisma error lets the request through. Since the auth fallback was narrowed to transient connectivity failures (is_database_connection_error allowlists httpx connection errors, EngineConnectionError and no_db_connection), query-level Prisma errors such as P2010 fail closed with a 401 because the database is reachable. A customer hit exactly that mismatch and asked what the intended blast radius is

This splits the row into "Prisma connection errors" (allowed, restricted INTERNAL_USER fallback identity) and "Prisma query errors" (blocked, 401), and adds a paragraph on what keeps working during an outage: cached virtual keys until user_api_key_cache_ttl expires, the master key and config-file models, while uncached key lookups, management endpoints and spend log writes wait for the database

Linear ticket

Resolves LIT-7857

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

…vers

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 16, 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 16, 2026 10:28am UTC

Request Review

@yassin-berriai
yassin-berriai merged commit 0f16943 into main Sep 17, 2026
4 checks passed
@yassin-berriai
yassin-berriai deleted the litellm_docs_db_unavailable_prisma_query_errors branch September 17, 2026 22:00
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