From ef877b7e700804201d8b859cd8b47307c8bb60fc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andr=C3=A9=20Noack?= <noack@data-quest.de>
Date: Mon, 21 Aug 2023 16:04:38 +0000
Subject: [PATCH] Resolve "SSO Login fragt getUser() ab, aber es gibt u.U.
 keinen Stud.IP Nutzer dazu"

Closes #3058

Merge request studip/studip!2045
---
 lib/phplib/Seminar_Auth.class.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/phplib/Seminar_Auth.class.php b/lib/phplib/Seminar_Auth.class.php
index dc65d19ef90..c628e0a2683 100644
--- a/lib/phplib/Seminar_Auth.class.php
+++ b/lib/phplib/Seminar_Auth.class.php
@@ -261,9 +261,8 @@ class Seminar_Auth
 
             // then do login
             if (($authplugin = StudipAuthAbstract::GetInstance($provider))) {
-                $authplugin->authenticateUser('', '');
-                if ($authplugin->getUser()) {
-                    $user = $authplugin->getStudipUser($authplugin->getUser());
+                $user = $authplugin->authenticateUser('', '');
+                if ($user) {
                     if ($user->isExpired()) {
                         throw new AccessDeniedException(_('Dieses Benutzerkonto ist abgelaufen. Wenden Sie sich bitte an die Administration.'));
                     }
@@ -279,6 +278,8 @@ class Seminar_Auth
                     Metrics::increment('core.sso_login.succeeded');
 
                     return $user->id;
+                } else {
+                    PageLayout::postMessage(MessageBox::error($authplugin->plugin_name . ': ' . _('Login fehlgeschlagen'), $authplugin->error_msg ? [$authplugin->error_msg] : []),md5($authplugin->error_msg));
                 }
             }
         }
-- 
GitLab