-
Notifications
You must be signed in to change notification settings - Fork 383
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
Registrations are poorly specified #1984
Comments
Anecdotally, synapse seems to just return a 403 when guest registration is disabled. (It also returns a 403 when a user tries to register with normal registration is disabled.) |
What error code would it return in both cases, though? As I understand it, Matrix error codes are prioritized over HTTP status codes to infer meaning from a response. |
In this case, Synapse seems to return |
Right. In that case, it should be safe to return an |
Currently, the documentation for the /register endpoint is pretty sparse about guest registrations. Specifically, I've found the following things to be missing so far:
password
is required #719 claims that passwords are optional for guest accounts. But are they permitted for a guest account?I guess that all of these points, except for the first one, can be summarized as it being unclear how the required-ness of fields is affected by the guest-ness of an account.
The text was updated successfully, but these errors were encountered: