Skip to content
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

Deprecate authentication via a query string #1808

Merged
merged 2 commits into from Apr 30, 2024
Merged

Conversation

zecakeh
Copy link
Contributor

@zecakeh zecakeh commented Apr 26, 2024

As per MSC4126.

It seems that OpenAPI 3.1 doesn't allow to mark a security scheme as deprecated, which is why I put it in the description.

Preview: https://pr1808--matrix-spec-previews.netlify.app

As per MSC4126

Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
@zecakeh zecakeh requested a review from a team as a code owner April 26, 2024 12:44
Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

Thanks!

@richvdh richvdh merged commit 625999a into matrix-org:main Apr 30, 2024
12 checks passed
@zecakeh zecakeh deleted the msc4126 branch April 30, 2024 16:10
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.

None yet

2 participants