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

Allow email registration, password reset & adding 3pids when no IS #10552

Closed
dbkr opened this issue Aug 14, 2019 · 6 comments

Comments

@dbkr
Copy link
Member

commented Aug 14, 2019

When the HS doesn't require an IS to send any of these validation tokens, riot should allow you to do all of these things when you have no IS configured.

This requires some spec to allow the client to know that it doesn't need to supply the id_server param to the various requestToken endpoints.

@anoadragon453

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

This requires some spec to allow the client to know that it doesn't need to supply the id_server param to the various requestToken endpoints.

Capabilities or somewhere else?

@dbkr

This comment has been minimized.

Copy link
Member Author

commented Aug 14, 2019

Yes. Capabilities currently requires auth though, so that's out for registration.

@anoadragon453

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

Maybe a GET /_matrix/client/r0/3pid/identity_server with a

{
    "identity_server": "example.com"
}

or

{
    "identity_server": null
}

response?

@dbkr

This comment has been minimized.

Copy link
Member Author

commented Aug 14, 2019

Travis was suggesting a param in the UI auth flow, so that's an option.

@anoadragon453

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

Ah, yeah that would be cleaner.

@dbkr

This comment has been minimized.

Copy link
Member Author

commented Aug 15, 2019

Closing in favour of split out bugs above

@dbkr dbkr closed this Aug 15, 2019

Workflow automation moved this from In Progress to In Test Aug 15, 2019

@dbkr dbkr removed this from In Test in Workflow Aug 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.