-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Extend password reset token validity on password change page load #25279
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
Extend password reset token validity on password change page load #25279
Conversation
This avoids a race condition where the password reset token is valid at page load time, but no longer valid at page submission time.
Hi @fredden. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
@magento create issue |
Hi @fredden, |
@magento run all tests |
app/code/Magento/Customer/Controller/Account/CreatePassword.php
Outdated
Show resolved
Hide resolved
app/code/Magento/User/Controller/Adminhtml/Auth/ResetPassword.php
Outdated
Show resolved
Hide resolved
app/code/Magento/Customer/Controller/Account/CreatePassword.php
Outdated
Show resolved
Hide resolved
app/code/Magento/Customer/Controller/Account/CreatePassword.php
Outdated
Show resolved
Hide resolved
@magento run Functional Tests EE, WebAPI Tests |
@magento run Functional Tests B2B, Functional Tests CE |
Hi @ihor-sviziev, thank you for the review. |
Note: Automation tests are passed |
@magento-engcom-team Any progress update here please? |
Hi @fredden, thank you for your contribution! |
Description (*)
This avoids a race condition where the password reset token is valid at page load time, but no longer valid at page submission time.
In a scenario where a user begins the password reset flow but is unable to immediately complete the same, clicking the link close to expiry time provides an opportunity for an error message after entering a new password due to the token expiring between page load & form submission.
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
I am unsure if this change requires a unit test.
Contribution checklist (*)
Resolved issues: