-
Notifications
You must be signed in to change notification settings - Fork 510
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
One step UserPass registration #388
Comments
This is one feature that we would like to have for our project as well. There is already a pull request with what you are looking for. Although there hasn't been any updates to it for a few months. |
@3x14159265 @lyuen My concern with making a one step registration is that we would be forced to show an error message if someone tries to register with an existing email address. This will leak information about the user base and could potentially be used by an attacker to target accounts knowing they exist. What are your thoughts on this? It might be depending on different needs/use cases that some people might be ok with that risk. |
I think there is no additional risk, the workflow should be the same as current one, i.e.:
|
I understand your concern @3x14159265. What @paiou says might work but it's a bit different to what #260 was doing. In it users are allowed to log in immediately (but marked as non verified). If you plan to support that then you will be forced to show an error if a sign up with an existing email address is attempted (again, depending on the user case people might be ok with this). @3x14159265 is your flow the same as what @paiou mentioned above? |
@jaliss However, allowing anonymous users to log-in was a feature I had in mind to implement next. We will have to accept the risk of leaking info then. |
First of all: great work on this project!
But: Is there any possibility to make the registration in one step, so that you can send only one form with email, name, password,.. ? I think the way it is done at the moment isn't a good approach in flavour of UX...
thanks!
The text was updated successfully, but these errors were encountered: