-
Notifications
You must be signed in to change notification settings - Fork 22
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
Forgot Password: Verification Link before reset #553
Comments
Additionally, changing the process for resets to limit User Enumeration attacks. Always send to "If account exists you will receive an email. https://cheatsheetseries.owasp.org/cheatsheets/Forgot_Password_Cheat_Sheet.html |
A two-step-verification for restting the password is a good idea, not sure how to implement this. |
I'd rather leave it as is. |
Well, we assume that the email addresses are correct in the database. We could implement a verification process in the context when accounts are created or email addresses are changed. The verification link for the changes should expire after a reasonable time span... There are a couple of details to evaluate carefully.
|
After digging in a little, the existing signup confirmation process may be useful to process the request. On signup the confirmcode and checksum are created (and updated). (account/confirm.php) The only portion that may need change is in the account tool (tool_account_settings/account/form_forgot.php) |
And since I am referencing tool_account_settings: I cant find an explanation, but how is this a module when it is part of core functions? Would it not make more since to be within the framework? (New to this level of interaction on an open-source project so please forgive me if already mentioned or explained.) |
It is a module because sometimes people want to make changes to the functionality of this part of the CMS and rather than letting them do in in the core part we decided to make it a module. |
Forgot Password autmatically resets the password of the user and does not require verification of email access.
This can be used to lock other users out by malicious actors.
Like the signup verify, allow for email verification before resetting the password.
The text was updated successfully, but these errors were encountered: