fix: use the Authenticator as entry point

Let the Authenticator implements the `AuthenticationEntryPoint` interface. (cherry picked from commit d60de33)
StudioAH · Feb 28, 2023 · 4b4d065 · 4b4d065
1 parent e6dae46
commit 4b4d065
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 42 deletions.
diff --git a/src/Security/CasAuthenticationEntryPoint.php b/src/Security/CasAuthenticationEntryPoint.php
diff --git a/src/Security/CasAuthenticator.php b/src/Security/CasAuthenticator.php
@@ -13,22 +13,28 @@
 
 use EcPhp\CasBundle\Cas\SymfonyCasInterface;
 use EcPhp\CasBundle\Security\Core\User\CasUserProviderInterface;
+use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
 use Symfony\Component\Security\Core\Exception\AuthenticationException;
 use Symfony\Component\Security\Core\User\UserInterface;
 use Symfony\Component\Security\Http\Authenticator\AbstractAuthenticator;
 use Symfony\Component\Security\Http\Authenticator\Passport\Badge\UserBadge;
 use Symfony\Component\Security\Http\Authenticator\Passport\Passport;
 use Symfony\Component\Security\Http\Authenticator\Passport\SelfValidatingPassport;
+use Symfony\Component\Security\Http\EntryPoint\AuthenticationEntryPointInterface;
 use Throwable;
 
-final class CasAuthenticator extends AbstractAuthenticator
+final class CasAuthenticator extends AbstractAuthenticator implements AuthenticationEntryPointInterface
 {
-    public function __construct(private readonly SymfonyCasInterface $cas, private readonly CasUserProviderInterface $userProvider)
-    {
+    public function __construct(
+        private readonly SymfonyCasInterface $cas,
+        private readonly CasUserProviderInterface $userProvider,
+        private readonly UrlGeneratorInterface $urlGenerator
+    ) {
     }
 
     public function authenticate(Request $request): Passport
@@ -81,6 +87,24 @@ public function onAuthenticationSuccess(Request $request, TokenInterface $token,
         return new RedirectResponse($request->getUri());
     }
 
+    public function start(Request $request, ?AuthenticationException $authException = null): Response
+    {
+        if (true === $request->isXmlHttpRequest()) {
+            return new JsonResponse(
+                ['message' => 'Authentication required'],
+                Response::HTTP_UNAUTHORIZED
+            );
+        }
+
+        return new RedirectResponse(
+            $this
+                ->urlGenerator
+                ->generate(
+                    'cas_bundle_login',
+                )
+        );
+    }
+
     public function supports(Request $request): bool
     {
         return $this