Defer updating of last_used_at

laravel · Dec 23, 2024 · 8a3d721 · 8a3d721
1 parent adc4b7d
commit 8a3d721
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/Guard.php b/src/Guard.php
@@ -80,12 +80,12 @@ public function __invoke(Request $request)
             if (method_exists($accessToken->getConnection(), 'hasModifiedRecords') &&
                 method_exists($accessToken->getConnection(), 'setRecordModificationState')) {
                 tap($accessToken->getConnection()->hasModifiedRecords(), function ($hasModifiedRecords) use ($accessToken) {
-                    $accessToken->forceFill(['last_used_at' => now()])->save();
+                    defer(fn() => $accessToken->forceFill(['last_used_at' => now()])->save())->always();
 
                     $accessToken->getConnection()->setRecordModificationState($hasModifiedRecords);
                 });
             } else {
-                $accessToken->forceFill(['last_used_at' => now()])->save();
+                defer(fn() => $accessToken->forceFill(['last_used_at' => now()])->save())->always();
             }
 
             return $tokenable;

diff --git a/tests/Feature/GuardTest.php b/tests/Feature/GuardTest.php
@@ -157,6 +157,7 @@ public function test_authentication_with_token_succeeds_if_expires_at_not_passed
             'expires_at' => now()->addMinutes(60),
         ]);
 
+        $this->withoutDefer();
         $returnedUser = $guard->__invoke($request);
 
         $this->assertEquals($user->id, $returnedUser->id);
@@ -187,6 +188,7 @@ public function test_authentication_is_successful_with_token_if_no_session_prese
             'name' => 'Test',
         ]);
 
+        $this->withoutDefer();
         $returnedUser = $guard->__invoke($request);
 
         $this->assertEquals($user->id, $returnedUser->id);
@@ -331,6 +333,7 @@ public function test_authentication_is_successful_with_token_in_custom_header()
             return $request->header('X-Auth-Token');
         });
 
+        $this->withoutDefer();
         $returnedUser = $guard->__invoke($request);
 
         $this->assertEquals($user->id, $returnedUser->id);