-
Notifications
You must be signed in to change notification settings - Fork 1.5k
SEP-1046: Client credentials flow for M2M without user interaction #1157
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
SEP-1046: Client credentials flow for M2M without user interaction #1157
Conversation
…emas, just in time load jose for client
commit: |
|
Subject to review cycle and any changes requested applied, leaving some notes:
|
…emove server side implementation
…stantinov/typescript-sdk into feature/sep-1046-client-credentials
| //case "client_credentials": | ||
|
|
||
| // Additional auth methods will not be added on the server side of the SDK. | ||
| case 'client_credentials': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just forward it to the upstream provider?
pcarleton
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome, this LGTM
Draft PR: Implements SEP-1046 (modelcontextprotocol/ext-auth#3, https://github.com/modelcontextprotocol/ext-auth/blob/main/specification/draft/oauth-client-credentials.mdx).
Motivation and Context
The Client Credentials flow enables machine-to-machine authentication without user interaction. This flow requires pre-registered client credentials, which are typically established out-of-band through administrative channels. Dynamic Client Registration is not used in this flow.
How Has This Been Tested?
Unit tests added.
Breaking Changes
Additional methods added on
Types of changes
Checklist
Additional context