move session set user id

Signed-off-by: Julien Veyssier <[email protected]>
nextcloud · Jan 13, 2025 · ab92df4 · ab92df4
1 parent 67fd4f1
commit ab92df4
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/Controller/LoginController.php b/lib/Controller/LoginController.php
@@ -453,6 +453,7 @@ public function code(string $state = '', string $code = '', string $scope = '',
 		// get user ID attribute
 		$uidAttribute = $this->providerService->getSetting($providerId, ProviderService::SETTING_MAPPING_UID, 'sub');
 		$userId = $idTokenPayload->{$uidAttribute} ?? null;
+		$this->session->set(self::USER_ID, $userId);
 		if ($userId === null) {
 			$message = $this->l10n->t('Failed to provision the user');
 			return $this->build403TemplateResponse($message, Http::STATUS_BAD_REQUEST, ['reason' => 'failed to provision user']);
@@ -502,7 +503,6 @@ public function code(string $state = '', string $code = '', string $scope = '',
 		}
 
 		$this->session->set(self::ID_TOKEN, $idTokenRaw);
-		$this->session->set(self::USER_ID, $user->getUID());
 
 		$this->logger->debug('Logging user in');