Add logout listener to the list of authentication listeners

scheb · scheb · commit 19727865a12e · 2020-06-19T16:25:51.000+02:00
so that it can be sorted in correctly by SortFirewallListenersPass
diff --git a/src/Symfony/Bundle/SecurityBundle/DependencyInjection/SecurityExtension.php b/src/Symfony/Bundle/SecurityBundle/DependencyInjection/SecurityExtension.php
@@ -395,6 +395,7 @@ private function createFirewall(ContainerBuilder $container, string $id, array $
                 'csrf_token_id' => $firewall['logout']['csrf_token_id'],
                 'logout_path' => $firewall['logout']['path'],
             ]);
+            $listeners[] = new Reference($logoutListenerId);
 
             // add default logout listener
             if (isset($firewall['logout']['success_handler'])) {
diff --git a/src/Symfony/Component/Security/Http/Firewall.php b/src/Symfony/Component/Security/Http/Firewall.php
@@ -52,38 +52,13 @@ public function onKernelRequest(RequestEvent $event)
 
         $authenticationListeners = $listeners[0];
         $exceptionListener = $listeners[1];
-        $logoutListener = $listeners[2];
 
         if (null !== $exceptionListener) {
             $this->exceptionListeners[$event->getRequest()] = $exceptionListener;
             $exceptionListener->register($this->dispatcher);
         }
 
-        // Authentication listeners are pre-sorted by SortFirewallListenersPass
-        $authenticationListeners = function () use ($authenticationListeners, $logoutListener) {
-            if (null !== $logoutListener) {
-                $logoutListenerPriority = $this->getListenerPriority($logoutListener);
-            }
-
-            foreach ($authenticationListeners as $listener) {
-                $listenerPriority = $this->getListenerPriority($listener);
-
-                // Yielding the LogoutListener at the correct position
-                if (null !== $logoutListener && $listenerPriority < $logoutListenerPriority) {
-                    yield $logoutListener;
-                    $logoutListener = null;
-                }
-
-                yield $listener;
-            }
-
-            // When LogoutListener has the lowest priority of all listeners
-            if (null !== $logoutListener) {
-                yield $logoutListener;
-            }
-        };
-
-        $this->callListeners($event, $authenticationListeners());
+        $this->callListeners($event, $authenticationListeners);
     }
 
     public function onKernelFinishRequest(FinishRequestEvent $event)
@@ -117,9 +92,4 @@ protected function callListeners(RequestEvent $event, iterable $listeners)
             }
         }
     }
-
-    private function getListenerPriority(object $logoutListener): int
-    {
-        return $logoutListener instanceof FirewallListenerInterface ? $logoutListener->getPriority() : 0;
-    }
 }
