From b3f2d2182e1b032f4bf41e6f7f3d8e17c9e0299c 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 --- C4/Auth.pm | 3 --- 1 file changed, 3 deletions(-) diff --git a/C4/Auth.pm b/C4/Auth.pm index 91c00923e5..839c85aef1 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -816,7 +816,6 @@ sub checkauth { _version_check( $type, $query ); # state variables - my $loggedin = 0; my $auth_state = 'failed'; my %info; my ( $userid, $cookie, $sessionID, $flags ); @@ -856,7 +855,6 @@ sub checkauth { -secure => ( C4::Context->https_enabled() ? 1 : 0 ), -sameSite => 'Lax', )); - $loggedin = 1; } elsif ( $emailaddress) { # the Google OpenID Connect passes an email address @@ -1204,7 +1202,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