-
-
Notifications
You must be signed in to change notification settings - Fork 117
Add OAuth 2.0 dynamic client registration #2148
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
Heavily based on the event-fields shortcode but allows to show schemas in the /data/schemas directory. Signed-off-by: Kévin Commaille <[email protected]>
As per MSC2966 Signed-off-by: Kévin Commaille <[email protected]>
Signed-off-by: Kévin Commaille <[email protected]>
3e3ef67
to
93e8ec2
Compare
content/client-server-api/_index.md
Outdated
##### Dynamic client registration flow | ||
|
||
To register, the client sends an HTTP `POST` request to the | ||
`registration_endpoint`, which can be found in the server metadata. The body of |
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.
todo: add a link to the metadata endpoint
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.
I could do it now and then we just need to merge #2147 before this?
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.
#2147 is now merged, so sure!
Signed-off-by: Kévin Commaille <[email protected]>
Signed-off-by: Kévin Commaille <[email protected]>
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 other than missing a link to metadata
Signed-off-by: Kévin Commaille <[email protected]>
Link added! |
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.
Perfect
Based on #2141.
As per MSC2966.
Pull Request Checklist
Preview: https://pr2148--matrix-spec-previews.netlify.app