-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Registration should also do .well-known discovery #8309
Comments
@jryans I assume this is the configuration side? Or is this some future UX that I probably forgot about? |
I believe this is about entering a full MXID ( On login, entering a full MXID triggers .well-known discovery and changes the server to the discovery result. On registration, we currently don't do the same thing. Since the login vs. registration UX looks very similar, entering a full MXID should also trigger .well-known discovery and change the server on registration as well. It's possible some Riot admins will want to lock down registration to only their default HS, but that's a separate concern from what should be possible on Riot by default. It should be possible to easily login and register to any HS from any Riot (assuming the Riot instance allows it via config, which it would by default). @lampholder Does that sound correct to you? |
bump @lampholder |
I've overthought the hell out of this, and I've finally convinced myself it falls into the category of "coming to a decision I have confidence in is more faff than it's worth". (The complexity fwiw exists mostly in the space of how we want users to experience onboarding, what we want to train our users about mxids and how much we want to encourage users typing their credentials in to riot-like things on the internet). I conclude: the symmetry would be nice to have, but the frequency with which users register and login itself is not symmetric, so I don't think implementing the .well-known lookup for registration is as valuable as it was for login. So I think we can do this and worry about config/default config later, but I don't see it as a high priority change. |
I somehow missed your reply, but it sounds like this is a p3 feature, so flagging it as that and moving on. |
Currently, we only do .well-known server discovery for the login flow. We should also do it on registration to give a similar UX to both flows.
The text was updated successfully, but these errors were encountered: