-
-
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
hs_url param is broken #9908
Comments
this is somewhat intentionally broken in develop due to the recent login/auth changes btw. |
I hadn't considered this - the intention is for people to provide links to https://riot.im/app (or other riot instances) and specify the default homeserver via GET params. In theory this sounds like a good thing to keep supporting - do we have any idea how well used it is? @turt2live is it possible to:
|
@lampholder the exact same parameters are heavily intertwined with email validation already, and are ignored by the validator for this reason |
Any update on this? We don't want to host yet another copy of Riot, but without the ability to redirect to homeserver login we're forced to. |
It's no easier to do than it was in May. Hosting your own Riot is going to be much better. |
Just found out that this is broken which is a bit annoying since I often times get issues with people onboarding to my home server since they immediately try to login to matrix.org instead. The solution "hosting your own Riot" doesn't make much sense to me because riot-web's Readme already mentions that hosting your own instance with the same domain as your home server brings in some security risks. |
@TheTimeWalker It's only an issue to host Riot on the same hostname as your homeserver, not demain. So if your Riot is on riot.example.com, and your homeserver is on synapse.example.com, then you're fine. |
Closing in favour of #5469 which has more upvotes |
users used to be able to set the default homeserver URL by setting a parameter ( matrix-org/matrix-react-sdk#716 ), but it looks like it was broken in the redesign
The text was updated successfully, but these errors were encountered: