-
Notifications
You must be signed in to change notification settings - Fork 788
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
[TO CLARIFY] Account Re-Verification on Email Change #705
Comments
Hmm interesting! This means the account would be inactive until the user confirms the new email. Question to everybody: How do big sites handle this ? |
Two email inputs.
This can be easiest. |
Good point! I see the following solution could be appropriate: Approach 1
Approach 2
Approach 3
Some general notes:
NOTE I'll keep this comment updated whenever something better, or if i found a security hole in the above steps. Updated 26/08/2015 9:57 GMT |
I think that two emails shouldn't be send the same time. If someone use verification link in new email first then will be to late to revoke changes.
|
Updated 26/08/2015 9:57 GMT It don't see a big difference, in both ways you are going to send email to new and old user's email one after another.Let me know if i missed something here. |
Btw if the user really mis-spelled the email adress then it's still possible to login with username. |
@panique that is spot on, It would simplify implementing this as well. |
As this is a good, but also a very advanced feature and would bloat the project quite much I would kindly close this ticket, but link to it from the readme, so people who really need this feature will still find this ticket. |
I was thinking...
If we use email verification for an account on creation, why would we not do the same when a user changes their email?
To me this only makes sense. It could go something like this...
Obviously there should be some sort of notification shown to the user alerting them that their account will be set back to inactive.
Thoughts?
Here is a quick setup for the function (Not tested)
The text was updated successfully, but these errors were encountered: