FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

SEP-2352: Clarify authorization server binding and migration by dend · Pull Request #2352 · modelcontextprotocol/modelcontextprotocol · GitHub

SEP-2352: Clarify authorization server binding and migration - #2352

Merged
mcp-commander[bot] merged 13 commits into
mainfrom
den/as
Mar 28, 2026
Merged

SEP-2352: Clarify authorization server binding and migration#2352
mcp-commander[bot] merged 13 commits into
mainfrom
den/as

Conversation

dend commented Mar 4, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • npx prettier --check passes
  • npm run check:docs:js-comments passes
  • Review rendered docs for correct formatting

🤖 Generated with Claude Code

dend requested a review from a team as a code owner March 4, 2026 22:42
Add guidance for clients on binding persisted client credentials to
specific authorization servers and handling AS changes. Clarifies
multi-AS behavior when multiple servers are listed in
authorization_servers.

Closes #1349

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
dend changed the title docs: clarify authorization server binding and migration SEP-2352: Clarify authorization server binding and migration Mar 4, 2026
dend self-assigned this Mar 4, 2026
dend added auth security SEP draft SEP proposal with a sponsor. labels Mar 4, 2026
dend moved this to In Review in SEP Review Pipeline Mar 4, 2026

pcarleton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This is a good change. Both typescript and python will need to adjustments.

Should be straightforward to conformance test as well, and key it on DCR support.

localden and others added 2 commits March 5, 2026 08:15
Co-authored-by: Kaixuan Luo <30545735+KevinLuo2000@users.noreply.github.com>
Co-authored-by: Kaixuan Luo <30545735+KevinLuo2000@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Max Gerber <89937743+max-stytch@users.noreply.github.com>
dend and others added 3 commits March 6, 2026 14:29
Co-authored-by: Wils Dawson <wils.dawson@gmail.com>
Co-authored-by: Wils Dawson <wils.dawson@gmail.com>
Co-authored-by: Wils Dawson <wils.dawson@gmail.com>
localden moved this from Accepted to Final in SEP Review Pipeline Aug 23, 2026
This was referenced Aug 23, 2026
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

Labels

Development

Successfully merging this pull request may close these issues.

The authorization spec is unclear when multiple Authorization servers are involved

10 participants


Back | FazBrowse Home | New Git URL