feat(mcp): backport the OBO token endpoint discovery stack (#31526, #31622, #31762) to 1.91.0rc1 - #32099
Closed
mateo-berri wants to merge 3 commits into
Closed
feat(mcp): backport the OBO token endpoint discovery stack (#31526, #31622, #31762) to 1.91.0rc1#32099mateo-berri wants to merge 3 commits into
mateo-berri wants to merge 3 commits into
Conversation
…no IdP guessing) (#31762) Backport of #31762 to patch-1.91.0rc1. Cherry-picked from c737789 (litellm_internal_staging). Conflict resolutions drop context belonging to PRs not on this line (#31809 byom tests, #31921's upstream absorb-variant of the list handler; this line keeps its own backported #31921 re-raise behavior).
mateo-berri
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relevant issues
Backports #31762 to the 1.91.0rc1 patch line. #31762 is stacked on #31622, which in turn builds on #31526; neither parent was on this line, so the full three-PR OBO stack comes along, one cherry-pick commit per PR with original authorship preserved
Linear ticket
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
@greptileaiand received a Confidence Score of at least 4/5 before requesting a maintainer reviewDelays in PR merge?
If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).
Screenshots / Proof of Fix
Tests were brought over as part of each cherry-pick; the full MCP/OBO battery on this branch passes locally: 564 passed across test_discoverable_endpoints.py, test_mcp_server_manager.py, test_mcp_stale_session.py, and outbound_credentials/. Live proof of the feature itself (real Keycloak IdP, discovery success and no-IdP-guess fail-closed) is on the source PRs #31622 and #31762
Type
🆕 New Feature
Changes
Conflict resolutions were kept minimal so this line only gains the OBO stack and nothing else that landed on staging since the rc cut. Specifically, hunks belonging to PRs not on this line were dropped during resolution: #31736's
_raise_unless_oauth2_discovery_serverdiscovery gating (this line keeps its existing fall-through for non-oauth2 servers), #31641's outbound-concurrency semaphore wrapper, and #31809's BYOM tests. In server.py, the list handler keeps this line's already-backported #31921 re-raise behavior rather than upstream's absorb variant; #31762's change there was comment-onlyNo version bump: rc lines keep pyproject at 1.91.0, the rc number is tag-only
Generated by Claude Code