-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Customer who exceeded max login failures not able to login even after reset password #15255
Comments
Magento Team, |
@tizzyguy87 there is a patch for this in #15534, can you check it and report back? |
Sure. I will work on it and provide an update. |
Hi @miguelbalparda, We are using Magento 2.2.0. I can see that you have added only 1 line of code as a fix. There is difference in original code in file which you have modified. Which version of Magento you used? As there is only 1 line of code, I added same line in my resetPassword method and it didn't fix the issue. I am still not able to login. Please find attached file from Magento 2.2.0 version with your fix at line number 556 Can you please check this. |
It seems some code you are missing was introduced in 2.2.3. I'd recommend upgrading to the latest 2.2 available and then apply the patch to see if it fixes your issue. |
I upgraded Magento to 2.2.4. But unfortunately we have different issue (#15664). So I am not able to login once user is created. I will verify fix you have suggested once that issue is resolved. Thanks. |
@miguelbalparda If I enter incorrect passwords, it is not updating failures_number and first_failure. Not sure how to fix this. |
Hi @tizzyguy87. Thank you for your report. The fix will be available with the upcoming 2.2.6 release. |
[Backport] magento#15255 unlock customer after password reset
[Forwardport] magento#15255 unlock customer after password reset
Hi @tizzyguy87. Thank you for your report. The fix will be available with the upcoming 2.3.0 release. |
Hi @tizzyguy87. Thank you for your report. The fix will be available with the upcoming 2.1.15 release. |
Has this really been fixed in 2.3? I am seeing the same issue and after checking the code against the linked commits, I am not seeing the $this->getAuthentication()->unlock($customer->getId()); line in the resetPassword function. |
Preconditions
Steps to reproduce
Expected result
Actual result
Other Details
I tried to set failure num to 0 in customer_entity and did password reset. Still customer is not allowed to login.
The text was updated successfully, but these errors were encountered: