Merge pull request #610 from leepeuker/fix-pwa-session-expiration

Update auth cookie settings to fix PWA losing cookie after restart
leepeuker · Aug 21, 2024 · e225ab2 · e225ab2
2 parents 0a764f3 + a8ebe96
commit e225ab2
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/src/Domain/User/Service/Authentication.php b/src/Domain/User/Service/Authentication.php
@@ -219,14 +219,13 @@ public function setAuthenticationCookieAndNewSession(int $userId, string $token,
     {
         $this->sessionWrapper->destroy();
         $this->sessionWrapper->start();
-        setcookie(self::AUTHENTICATION_COOKIE_NAME, $token, [
-            'expires' => (int)$expirationDate->format('U'),
-            'path' => '/',
-            'domain' => '',
-            'secure' => false,
-            'httponly' => true,
-            'samesite' => 'strict'
-        ]);
+        setcookie(
+            self::AUTHENTICATION_COOKIE_NAME,
+            $token,
+            (int)$expirationDate->format('U'),
+            '/',
+            httponly: true
+        );
 
         $this->sessionWrapper->set('userId', $userId);
     }