From 3e9a24232c4e6945996ec056db4b60bdf3628364 Mon Sep 17 00:00:00 2001
From: Binyamin Yawitz <316103+byawitz@users.noreply.github.com>
Date: Mon, 7 Oct 2024 15:26:01 -0400
Subject: [PATCH] fix: introducing local variable

---
 app/controllers/api/account.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/controllers/api/account.php b/app/controllers/api/account.php
index a76b9dbea5..007435a5df 100644
--- a/app/controllers/api/account.php
+++ b/app/controllers/api/account.php
@@ -1379,6 +1379,7 @@ Http::get('/v1/account/sessions/oauth2/:provider/redirect')
         $email = $oauth2->getUserEmail($accessToken);
 
         // Check if this identity is connected to a different user
+        $sessionUpgrade = false;
         if (!$user->isEmpty()) {
             $userId = $user->getId();
 
@@ -1672,7 +1673,7 @@ Http::get('/v1/account/sessions/oauth2/:provider/redirect')
                 ->addCookie($authentication->getCookieName(), Auth::encodeSession($user->getId(), $secret), (new \DateTime($expire))->getTimestamp(), '/', Config::getParam('cookieDomain'), ('https' == $protocol), true, Config::getParam('cookieSamesite'));
         }
 
-        if (isset($sessionUpgrade) && $sessionUpgrade) {
+        if ($sessionUpgrade) {
             foreach ($user->getAttribute('targets', []) as $target) {
                 if ($target->getAttribute('providerType') !== MESSAGE_TYPE_PUSH) {
                     continue;