repair Shibboleth login fixes #5045

Closes #5045

Merge request studip/studip!3810

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/authentication/Manager.php b/lib/authentication/Manager.php
index d9c9f95..b25efad 100644
--- a/lib/authentication/Manager.php
+++ b/lib/authentication/Manager.php
@@ -55,7 +55,7 @@ class Manager
                 Metrics::increment('core.sso_login.attempted');
                 // then do login
                 $authplugin = StudipAuthAbstract::GetInstance($provider);
-                if ($authplugin) {
+                if ($authplugin instanceof \StudipAuthSSO) {
                     $authplugin->authenticateUser('', '');
                     if ($authplugin->getUser()) {
                         $user = $authplugin->getStudipUser($authplugin->getUser());
@@ -71,8 +71,8 @@ class Manager
                         }
                         Metrics::increment('core.sso_login.succeeded');
 
-                        sess()->regenerateId(['auth', '_language', 'phpCAS', 'contrast']);
                         $this->setAuthenticatedUser($user);
+                        sess()->regenerateId(['auth', '_language', 'phpCAS', 'contrast']);
                     }
                 }
             }