Throw Exception when logout handler is called

Author: alexander-schranz

src/Security/Http/Logout/ContextInvalidationLogoutHandler.php

@@ -15,6 +15,7 @@
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
+use Symfony\Component\Security\Http\Event\LogoutEvent;
 use Symfony\Component\Security\Http\Logout\LogoutHandlerInterface;
 
 /**
@@ -38,6 +39,13 @@ public function __construct(UserContextInvalidator $invalidator)
     public function logout(Request $request, Response $response, TokenInterface $token)
     {
         @trigger_error('Using the ContextInvalidationLogoutHandler is deprecated', E_USER_DEPRECATED);
+
+        if (class_exists(LogoutEvent::class)) {
+            // This function should not longer be called in Symfony 5.1
+            // See also: https://github.com/FriendsOfSymfony/FOSHttpCacheBundle/pull/545#discussion_r465089219
+            throw new \LogicException(__CLASS__ . '::' . __METHOD__ . ' is deprecated and should since Symfony 5.1 not longer be called.');
+        }
+
         $this->invalidator->invalidateContext($request->getSession()->getId());
     }
 }

tests/Functional/Security/Http/Logout/ContextInvalidationLogoutHandlerTest.php

@@ -26,6 +26,7 @@ public function testLogout()
     {
         if (class_exists(LogoutEvent::class)) {
             // @see https://github.com/symfony/symfony/pull/36243/files#r465083756
+            // @see https://github.com/FriendsOfSymfony/FOSHttpCacheBundle/pull/545/files#diff-05cbcfd492fd361b33ee70130dc687b2
             $this->markTestSkipped('This test does not work with Symfony 5.1.');
         }