docs(mcp): document delegated OAuth admission migration - #1449
joshua-berri wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@greptileai Please review this MCP migration documentation for accurate authentication boundaries, unchanged upstream token handling, and the upcoming breaking-change notice. |
|
@cursor review Please check this docs-only migration for incorrect authentication claims, broken examples, and regressions in existing MCP documentation links. |
|
@veria-ai Please review the security wording around legacy delegated admission, public OAuth discovery, and the distinction between authentication and unattributed spend. |
There was a problem hiding this comment.
✅ 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 3c59a88. Configure here.
Summary
Document the removal of anonymous admission for legacy delegated OAuth MCP routes. The upstream OAuth token still uses
Authorization: Bearer <upstream-token>unchanged; LiteLLM admission is now required separatelyThe migration guide recommends
oauth_delegate, preserves the existing legacy section anchor, shows the two-header key flow, and links OAuth-only clients to the existing authenticated DCR bridge. Public discovery is distinguished from permission to open sessions or invoke toolsThe cost and public-internet pages clarify unattributed passthrough spend, the absence of identity-based budget debits, and the separate IP access controls
Breaking-change placement
The fix merged after the currently published releases. As agreed, an upcoming red
Breaking Changesnotice lives in the MCP guide, explicitly scoped to builds containing the backend fix. This PR does not assign a version or alter an older release noteWhen the containing release is assigned, its release note should include the warning below the deployment block, following the repository convention
Scope
Only
docs/mcp_oauth_passthrough.md,docs/mcp_public_internet.md, anddocs/mcp_cost.mdchange. No new pages, screenshots, dependency changes, instruction files, or unrelated formattingValidation
npm run lint:writingpassed across 1,044 Markdown files; its nine existing vocabulary warnings are outside the edited pagespython3 scripts/check-docs.py docspassed across 800 docs pages, including links, anchors, and embedded examplesnpm run buildpassed with the locked dependencies and Node 20, matching the CI major versionGenerated HTML for all three edited pages was checked: the notice renders with danger styling, the legacy migration anchor is preserved, the authenticated bridge and cost-attribution links resolve, and the two-token curl example is present
git diff --checkpassed. Validation applies to commit3c59a882. Hosted writing-style and structure checks passed; the deployment and requested bot reviews are still pendingThe Vercel preview requires Vercel sign-in. Generated pages were verified locally without starting a server or requesting browser access
This is documentation-only; there is no changed executable behavior requiring Python regression coverage
Existing environment warnings
npm cireported 64 vulnerabilities in the unchanged dependency lockfile. The successful build also reports existing blog-preview, browser-data, and HTML-minifier warnings on other pages. No edited MCP page appears in those diagnostics; no unrelated content was changed and no checks were suppressedReferences
Closes #1448
Backend fix: BerriAI/litellm#40923
Linear: https://linear.app/litellm-ai/issue/LIT-6970/mcp-tool-calls-on-passthrough-upstream-delegated-servers-are-recorded