diff --git a/src/Security/MemberAuthenticator/ChangePasswordHandler.php b/src/Security/MemberAuthenticator/ChangePasswordHandler.php index 5aefa310e38..4a6c1c3c8b9 100644 --- a/src/Security/MemberAuthenticator/ChangePasswordHandler.php +++ b/src/Security/MemberAuthenticator/ChangePasswordHandler.php @@ -13,6 +13,7 @@ use SilverStripe\ORM\ValidationException; use SilverStripe\Security\Authenticator; use SilverStripe\Security\IdentityStore; +use SilverStripe\Security\LoginAttempt; use SilverStripe\Security\Member; use SilverStripe\Security\Security; @@ -119,8 +120,8 @@ public function changepassword() _t( 'SilverStripe\\Security\\Security.NOTERESETLINKINVALID', '

The password reset link is invalid or expired.

' - . '

You can request a new one here or change your password after' - . ' you log in.

', + . '

You can request a new one here or change your password after' + . ' you log in.

', [ 'link1' => Security::lost_password_url(), 'link2' => Security::login_url(), @@ -267,11 +268,24 @@ public function doChangePassword(array $data, $form) // Clear locked out status $member->LockedOutUntil = null; $member->FailedLoginCount = null; + // Clear the members login hashes $member->AutoLoginHash = null; $member->AutoLoginExpired = DBDatetime::create()->now(); $member->write(); + // Create a successful 'LoginAttempt' as the password is reset + $loginAttempt = LoginAttempt::create(); + $loginAttempt->Status = LoginAttempt::SUCCESS; + $loginAttempt->MemberID = $member->ID; + + if ($member->Email) { + $loginAttempt->setEmail($member->Email); + } + + $loginAttempt->IP = $this->getRequest()->getIP(); + $loginAttempt->write(); + if ($member->canLogin()) { $identityStore = Injector::inst()->get(IdentityStore::class); $identityStore->logIn($member, false, $this->getRequest()); @@ -279,9 +293,10 @@ public function doChangePassword(array $data, $form) $session->clear('AutoLoginHash'); - // Redirect to backurl + // Redirect to back url $backURL = $this->getBackURL(); - if ($backURL + if ( + $backURL // Don't redirect back to itself && $backURL !== Security::singleton()->Link('changepassword') ) {