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

Resetting Password: Automatically logs user in even if the account isn't confirmed. #38

Open
ACPK opened this issue Aug 10, 2015 · 6 comments
Labels

Comments

@ACPK
Copy link

ACPK commented Aug 10, 2015

When a user resets their password, it automatically logs them into their account (i.e. doesn't check if their account is confirmed). If they log out and then try to later sign in, they get the message "Account not confirmed yet. Please follow the instructions we sent you by email."

@manukall
Copy link
Owner

that's a bug. maybe we should just confirm the users account when they reset their password, as we know they entered a correct email.

@manukall manukall added the bug label Aug 10, 2015
@ACPK
Copy link
Author

ACPK commented Aug 10, 2015

@manukall That sounds awesome!

@clekstro
Copy link
Contributor

I'm just evaluating this as a possible solution for an upcoming Phoenix project (so apologies for the armchair quarterbacking), but the proposed solution feels odd. If I haven't confirmed my email, it seems strange that I could log in at all. Just because I've entered an email, doesn't mean it's mine. That's why the confirmation is required in the first place.

In practice, doesn't this mean that anyone who's able to guess an unconfirmed account's email is able to log in as that user?

@ACPK
Copy link
Author

ACPK commented Aug 30, 2015

@clekstro In order to reset your password, you need to get the confirmation email. Hence, they (1) submit for password reset form, (2) click the link that gets emailed to them, and (3) get brought to a page where they change their password.

BTW, in regards to logging in when your email isn't confirmed, a few websites where you're automatically logged in before confirming your email are Github, Facebook, LinkedIn, Twitter. etc.

@clekstro
Copy link
Contributor

@ACPK Appreciate the additional context. I didn't realize from the description above that that was the flow.

@ACPK
Copy link
Author

ACPK commented Aug 30, 2015

@clekstro No problem. We're using @manukall so I recommend it but I'm adding some custom features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants