Skip to content

Commit 23ce54e

Browse files
committed
Make sure we’re saving the last_login in 2FA auths
Signed-off-by: snipe <[email protected]>
1 parent 30c4e9d commit 23ce54e

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

app/Http/Controllers/Auth/LoginController.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,7 @@ private function loginViaLdap(Request $request): User
206206
$user->password = bcrypt($request->input('password'));
207207
}
208208

209+
$user->last_login = \Carbon::now();
209210
$user->email = $ldap_attr['email'];
210211
$user->first_name = $ldap_attr['firstname'];
211212
$user->last_name = $ldap_attr['lastname']; //FIXME (or TODO?) - do we need to map additional fields that we now support? E.g. country, phone, etc.
@@ -432,6 +433,7 @@ public function postTwoFactorAuth(Request $request)
432433

433434
if (Google2FA::verifyKey($user->two_factor_secret, $secret)) {
434435
$user->two_factor_enrolled = 1;
436+
$user->last_login = \Carbon::now();
435437
$user->saveQuietly();
436438
$request->session()->put('2fa_authed', $user->id);
437439

0 commit comments

Comments
 (0)