Fix registration of failure handler

hwi · Feb 16, 2024 · 90c9016 · 90c9016
1 parent 2b94931
commit 90c9016
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/DependencyInjection/Security/Factory/OAuthAuthenticatorFactory.php b/src/DependencyInjection/Security/Factory/OAuthAuthenticatorFactory.php
@@ -135,6 +135,18 @@ public function getFirewallNames(): \ArrayIterator
         return $this->firewallNames;
     }
 
+    protected function createAuthenticationFailureHandler(ContainerBuilder $container, string $id, array $config): string
+    {
+        $id = $this->getFailureHandlerId($id);
+        $options = array_intersect_key($config, $this->defaultFailureHandlerOptions);
+
+        $failureHandler = $container->setDefinition($id, new ChildDefinition('security.authentication.custom_failure_handler'));
+        $failureHandler->replaceArgument(0, new ChildDefinition('hwi_oauth.authentication.failure_handler'));
+        $failureHandler->replaceArgument(1, $options);
+
+        return $id;
+    }
+
     /**
      * {@inheritdoc}
      */

diff --git a/src/Resources/config/oauth.php b/src/Resources/config/oauth.php
@@ -62,5 +62,10 @@
         ]);
 
     $services->set('hwi_oauth.authentication.failure_handler', AuthenticationFailureHandler::class)
-        ->arg('$connect', '%hwi_oauth.connect%');
+        ->args([
+            service('request_stack'),
+            service('router'),
+            service('twig'),
+            '%hwi_oauth.connect%'
+        ]);
 };