Skip to content

Conversation

@m-henderson
Copy link
Contributor

Advertise none in token_endpoint_auth_methods_supported so that public OAuth clients can interoperate with the MCP Auth Router.

Motivation and Context

Dynamic client registration and token exchange flows for public clients require the AS metadata to list "none" as a supported token endpoint authentication method. Without this, public clients cannot complete an authorization code token flow.

This change updates the Authorization Server metadata to include "none" alongside "client_secret_post".

Fixes #1027

How Has This Been Tested?

  • Updated tests in src/server/auth/router.test.ts to expect both "client_secret_post" and "none".
  • All tests pass locally.
  • Verified manually using:
    • curl:
      curl -s http://localhost:port/.well-known/oauth-authorization-server \
        | jq '.token_endpoint_auth_methods_supported'
      # -> ["client_secret_post", "none"]

Breaking Changes

Not that im aware of. Test are passing but I do not have enough knowledge yet to verify backwards compatibility.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

I checked to see if there was specific documentation that needed to be updated but I didn't see any. If I need to add it let me know.

@felixweinberger Thanks for adding the good first issue label.

@m-henderson m-henderson requested a review from a team as a code owner November 15, 2025 05:48
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 15, 2025

Open in StackBlitz

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/sdk@1116

commit: 54b74b0

Copy link
Contributor

@mattzcarey mattzcarey left a comment

Choose a reason for hiding this comment

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

@pcarleton will need your stamp on this :)

@pcarleton pcarleton enabled auto-merge (squash) November 18, 2025 15:33
@pcarleton pcarleton merged commit 7e44b10 into modelcontextprotocol:main Nov 18, 2025
5 checks passed
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.

token_endpoint_auth_methods_supported should also support none

3 participants