BUGFIX since upgrade auth0 lib we are missing the login params

cultuurnet · Oct 1, 2024 · a544556 · a544556
1 parent d8cdfcb
commit a544556
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/app/Auth/AuthControllerProvider.php b/app/Auth/AuthControllerProvider.php
@@ -15,13 +15,15 @@ public function connect(Application $app): ControllerCollection
     {
         $app['auth_controller'] = $app->share(
             function (Application $app) {
+                $authConfig = $app['config']['keycloak']['enable'] ? $app['config']['keycloak'] : $app['config']['auth0'];
+
                 return new AuthController(
                     $app[Auth0::class],
                     $app['session'],
                     $app['uitid_user_session_service'],
                     $app[UiTIDv1TokenService::class],
-                    $app['config']['auth0']['login_parameters'],
-                    $app['config']['auth0']['app_url']
+                    $authConfig['login_parameters'],
+                    $authConfig['app_url']
                 );
             }
         );

diff --git a/src/Auth/AuthController.php b/src/Auth/AuthController.php
@@ -79,7 +79,7 @@ public function redirectToLoginService(Request $request): RedirectResponse
             $this->session->set('auth_destination', $destination);
         }
 
-        return new RedirectResponse($this->auth0->login(null, null, $this->loginParameters));
+        return new RedirectResponse($this->auth0->login(null, $this->loginParameters));
     }
 
     public function storeTokenAndRedirectToFrontend(): RedirectResponse