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.

Sign up for GitHub

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

Jump to bottom

Email notification upon password change #673

Closed
jholdstock opened this issue Jan 22, 2019 · 1 comment · Fixed by #680
Closed

Email notification upon password change #673

jholdstock opened this issue Jan 22, 2019 · 1 comment · Fixed by #680

Comments

@jholdstock
Copy link
Member

An email notification should be sent when the password on an account is changed.

One example for why this is important - presently somebody may leave their account logged in on an unattended machine with their password saved in their browser auto-complete. A malicious actor could change the password on the account without the account holder being aware that their account has been compromised.

Sending a notification upon password change is a standard security practice implemented by many websites - it does not prevent this scenario, but it does at least inform the account holder that their account is compromised.

(This is a suggestion from a report submitted to https://bounty.decred.org)
@jholdstock jholdstock mentioned this issue Jan 22, 2019
Closed
@thi4go
Copy link
Member

thi4go commented Jan 22, 2019

I'll work on this!

@thi4go thi4go mentioned this issue Jan 28, 2019
Merged
vibros68 pushed a commit to vibros68/politeia that referenced this issue Aug 17, 2021
@victorgcramos @tiagoalvesdulce
Fix Malformed URLs (decred#673)
cce6cd2 
* allow node function to check if link is valid when typing

* url validator

* added url validator for malformed links

* removed broken methods

* fix validation error on regular inputs

* added proposal link validator on sumbit proposal

* fixed errors

* fixed error when typing valid urls

* removed console.logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

www: Sends email notification when password is changed thi4go/politeia
2 participants
@thi4go @jholdstock