From d5c8cb1f2162a2304d64d19de1e0ac04ddb55159 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 24 Jan 2024 16:24:51 +0100 Subject: [PATCH] Bug 35890: Remove var loggedin It is never used and add confusion Signed-off-by: Matt Blenkinsop Signed-off-by: Kyle M Hall Signed-off-by: Wainui Witika-Park --- C4/Auth.pm | 3 --- 1 file changed, 3 deletions(-) diff --git a/C4/Auth.pm b/C4/Auth.pm index 8adbc78173..edc5e3ba64 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -847,7 +847,6 @@ sub checkauth { _version_check( $type, $query ); # state variables - my $loggedin = 0; my $auth_state = 'failed'; my %info; my ( $userid, $cookie, $sessionID, $flags ); @@ -884,7 +883,6 @@ sub checkauth { -secure => ( C4::Context->https_enabled() ? 1 : 0 ), -sameSite => 'Lax', )); - $loggedin = 1; } elsif ( $emailaddress) { # the Google OpenID Connect passes an email address @@ -1228,7 +1226,6 @@ sub checkauth { my $domain = $branches->{$branchcode}->{'branchip'}; $domain =~ s|\.\*||g; if ( $ip !~ /^$domain/ ) { - $loggedin = 0; $cookie = $cookie_mgr->replace_in_list( $cookie, $query->cookie( -name => 'CGISESSID', -value => '', -- 2.39.5