-
-
Notifications
You must be signed in to change notification settings - Fork 117
Add the OAuth 2.0 server metadata discovery endpoint #2147
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
Conversation
As per MSC2965. Signed-off-by: Kévin Commaille <[email protected]>
Signed-off-by: Kévin Commaille <[email protected]>
eb7db56
to
693e9f7
Compare
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.
A few nits
prompt_values_supported: | ||
type: array | ||
description: |- | ||
List of OpenID Connect prompt values that the server supports at the | ||
authorization endpoint. | ||
|
||
Only the `create` value defined in [Initiating User Registration via OpenID | ||
Connect](https://openid.net/specs/openid-connect-prompt-create-1_0.html) is | ||
supported, for a client to signal to the server that the user desires to | ||
register a new account. | ||
items: | ||
type: string | ||
description: A prompt value that the server supports. |
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.
For the historical record: this appears to have come from MSC2964.
Signed-off-by: Kévin Commaille <[email protected]>
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.
LGTM, but there is a conflict now
Signed-off-by: Kévin Commaille <[email protected]>
I merged main into my branch to solve the conflict, placing "Scope" after "Server metadata discovery". |
Thanks! |
Based on #2141.
As per MSC2965.
Pull Request Checklist
Preview: https://pr2147--matrix-spec-previews.netlify.app