Fix group evaluation

TYPO3GmbH · Aug 26, 2024 · 293e6db · 293e6db
1 parent 6fef605
commit 293e6db
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/Security/KeyCloakAuthenticator.php b/src/Security/KeyCloakAuthenticator.php
@@ -1,5 +1,5 @@
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /*
  * This file is part of the package t3g/symfony-keycloak-bundle.

diff --git a/src/Security/KeyCloakUserProvider.php b/src/Security/KeyCloakUserProvider.php
@@ -1,5 +1,5 @@
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /*
  * This file is part of the package t3g/symfony-keycloak-bundle.
@@ -18,7 +18,6 @@ class KeyCloakUserProvider implements UserProviderInterface
 {
     private array $roleMapping;
     private array $defaultRoles;
-    private array $userRoles = [];
 
     public function __construct(array $roleMapping, array $defaultRoles = ['ROLE_USER', 'ROLE_OAUTH_USER'])
     {
@@ -45,7 +44,7 @@ public function loadUserByIdentifier(
     ): KeyCloakUser {
         $roles = array_intersect_key($this->roleMapping, array_flip(array_map(static function ($v) {
             return str_replace('-', '_', $v);
-        }, $this->userRoles)));
+        }, $keycloakGroups)));
         $roles = array_merge($roles, $scopes, $this->defaultRoles);
 
         return new KeyCloakUser($identifier, array_values($roles), $email, $fullName, $fresh);