From 243efa7ee9a59de76c739d636d10e287da97b940 Mon Sep 17 00:00:00 2001 From: Arthur Suzuki Date: Fri, 13 Jan 2023 14:55:49 +0100 Subject: [PATCH] Revert "Bug 32208: (follow-up) Fix compile issue" This reverts commit 610644ae89b97ee27864eaff8d0f16f351356ea9. Signed-off-by: Arthur Suzuki --- C4/Auth.pm | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/C4/Auth.pm b/C4/Auth.pm index 06463e42d6..15d6ba362e 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -1109,10 +1109,13 @@ sub checkauth { $loggedin = 1; } else { + $auth_state = 'failed'; + # FIXME We could add $return = 0; or even delete the session? + # Currently return == 1 and we will fill session info later on, + # although we do present an authorization failure. (Yes, the + # authentication was actually correct.) $info{'nopermission'} = 1; C4::Context::_unset_userenv($sessionID); - $sessionID = undef; - $userid = undef; } my ( $borrowernumber, $firstname, $surname, $userflags, $branchcode, $branchname, $emailaddress, $desk_id, -- 2.39.2