Repository navigation
fix(client): invalidate credentials and re-register when the authorization server changes (SEP-2352) #2271
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
+1,574
−61
Closed
fix(client): invalidate credentials and re-register when the authorization server changes (SEP-2352) #2271
Changes from 14 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
66edf5f
fix(client): invalidate credentials and re-register when the authoriz…
mattzcarey 951c6d4
fix(client): refresh challenged discovery before AS binding
mattzcarey 8769208
fix(client): avoid false AS-change invalidation
mattzcarey 849bc81
fix(client): keep cached discovery after failed challenge
mattzcarey 6736204
fix(client): preserve authoritative AS discovery
mattzcarey a6534d5
fix(client): avoid cached AS false invalidation
mattzcarey 28bf20e
fix(client): preserve cached AS metadata on rediscovery
mattzcarey 7146f9c
fix(client): preserve AS-bound client during code exchange
mattzcarey 2b933a6
fix(client): persist refreshed resource metadata fallback
mattzcarey 16c69ab
fix(client): repair sep-2352 rebase
mattzcarey 705548f
fix(docs): clarify authorization server fallback hooks
mattzcarey 8cbf10a
fix(docs): clarify sep-2352 legacy url hooks
mattzcarey 7a45144
fix(client): discard unstamped credentials after AS change
mattzcarey 437d922
fix(client): preserve portable clients across AS changes
mattzcarey c320494
fix(client): gate portable client restamping
mattzcarey 3c02b02
fix(client): reuse cached discovery on code exchange
mattzcarey d05528b
fix(client): avoid step-up discovery refresh without fresh prm
mattzcarey 4ecaf7d
fix(client): read stale client with issuer context
mattzcarey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| '@modelcontextprotocol/client': patch | ||
| --- | ||
|
|
||
| Implement SEP-2352 authorization server binding: when OAuth discovery shows the authorization server has changed since client credentials were recorded, `auth()` now invalidates the stale client registration and tokens (`invalidateCredentials('client')` / `('tokens')`) and re-registers with the new authorization server. CIMD (HTTPS URL) client IDs are portable across authorization servers, so they are exempt from client re-registration, but their tokens are still invalidated when the authorization server changes. Provider implementations should persist client credentials keyed by the authorization server's `issuer` identifier. | ||
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.