fix(Realms): Fixed realm cache-key user awareness

roadiz · Mar 7, 2024 · ce71706 · ce71706
1 parent 4c46346
commit ce71706
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/lib/RoadizCoreBundle/src/Realm/RealmResolver.php b/lib/RoadizCoreBundle/src/Realm/RealmResolver.php
@@ -12,18 +12,15 @@
 use RZ\Roadiz\CoreBundle\Security\Authorization\Voter\RealmVoter;
 use Symfony\Component\HttpKernel\Exception\UnauthorizedHttpException;
 use Symfony\Component\Security\Core\Security;
+use Symfony\Component\String\Slugger\AsciiSlugger;
 
 final class RealmResolver implements RealmResolverInterface
 {
-    private ManagerRegistry $managerRegistry;
-    private Security $security;
-    private CacheItemPoolInterface $cache;
-
-    public function __construct(ManagerRegistry $managerRegistry, Security $security, CacheItemPoolInterface $cache)
-    {
-        $this->managerRegistry = $managerRegistry;
-        $this->security = $security;
-        $this->cache = $cache;
+    public function __construct(
+        private readonly ManagerRegistry $managerRegistry,
+        private readonly Security $security,
+        private readonly CacheItemPoolInterface $cache
+    ) {
     }
 
     public function getRealms(?Node $node): array
@@ -49,9 +46,16 @@ public function denyUnlessGranted(RealmInterface $realm): void
         }
     }
 
+    private function getUserCacheKey(): string
+    {
+        return (new AsciiSlugger())
+            ->slug($this->security->getUser()?->getUserIdentifier() ?? 'anonymous')
+            ->__toString();
+    }
+
     public function getGrantedRealms(): array
     {
-        $cacheItem = $this->cache->getItem('granted_realms');
+        $cacheItem = $this->cache->getItem('granted_realms_' . $this->getUserCacheKey());
         if (!$cacheItem->isHit()) {
             $allRealms = $this->managerRegistry->getRepository(Realm::class)->findBy([]);
             $cacheItem->set(array_filter($allRealms, fn(RealmInterface $realm) => $this->isGranted($realm)));
@@ -63,7 +67,7 @@ public function getGrantedRealms(): array
 
     public function getDeniedRealms(): array
     {
-        $cacheItem = $this->cache->getItem('denied_realms');
+        $cacheItem = $this->cache->getItem('denied_realms_' . $this->getUserCacheKey());
         if (!$cacheItem->isHit()) {
             $allRealms = $this->managerRegistry->getRepository(Realm::class)->findBy([]);
             $cacheItem->set(array_filter($allRealms, fn(RealmInterface $realm) => !$this->isGranted($realm)));