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

Require users to input their password to change their email #1651

Closed
ToksT opened this issue May 21, 2013 · 3 comments
Closed

Require users to input their password to change their email #1651

ToksT opened this issue May 21, 2013 · 3 comments
Labels

Comments

@ToksT
Copy link
Contributor

ToksT commented May 21, 2013

Emails can be used for password resets.

If someone got access to your account via another vulnerability like #886, they wouldn't normally know your password, so they would only temporarily have access.

But if they could change your email, they could change your password via a reset, thus gaining permanent access. (Also opening up the ability to delete the account, since they now have the password.)

This could be avoided by requiring the password to be input when changing email.

@ghostrigger
Copy link
Contributor

in case we want to step it up a bit further maybe require the user to enter password when accessing / changing Settings this will protect both the delete and email user settings. or for more paranoid type another confirmation when deleting?

@MyrMindservant
Copy link

in case we want to step it up a bit further maybe require the user to enter password when accessing / changing Settings this will protect both the delete and email user settings.

This is overdoing it. I don't want to enter my password every time I change something there. Don't forget that blacklisted tags and frequently used tags are also at the settings page and they can be modified fairly frequently.

@ToksT
Copy link
Contributor Author

ToksT commented May 21, 2013

I agree with @MyrMindservant. I'm not sure anyone who gains access to your account would have reason to change your other settings, but if they did it shouldn't have irreversible consequences. Email/password/deletion should be enough to ask for confirmation.

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