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

MSC4128: Error on invalid auth where it is optional #4128

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Kladki
Copy link

@Kladki Kladki commented Apr 11, 2024

Rendered

Implementations:

Fixes matrix-org/matrix-spec#1779

@Kladki Kladki force-pushed the error-optional-auth branch 2 times, most recently from 1ffa3c6 to 4b9bb1d Compare April 11, 2024 12:41
Signed-off-by: Matthias Ahouansou <matthias@ahouansou.cz>
@turt2live turt2live added proposal A matrix spec change proposal client-server Client-Server API kind:maintenance MSC which clarifies/updates existing spec needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. labels Apr 11, 2024
Copy link
Member

Choose a reason for hiding this comment

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

Implementation requirements:

@turt2live turt2live removed the needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. label Apr 11, 2024
@Kladki Kladki marked this pull request as draft April 15, 2024 17:37
@Kladki Kladki marked this pull request as ready for review April 15, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client-server Client-Server API kind:maintenance MSC which clarifies/updates existing spec proposal A matrix spec change proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It's not clear whether /versions should fail if auth is invalid
2 participants