From 64b3fb0782c3f019725ac3a49ae97180eabf3662 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 24 Sep 2024 15:22:34 +0200 Subject: [PATCH] Bug 37041: Store session_id in userenv --- C4/Auth.pm | 31 ++++--------------------------- 1 file changed, 4 insertions(+), 27 deletions(-) diff --git a/C4/Auth.pm b/C4/Auth.pm index d4b83bb75ea..ef7fe2999ae 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -1289,15 +1289,7 @@ sub checkauth { $session->param( 'sco_user', $is_sco_user ); } $session->param('cas_ticket', $cas_ticket) if $cas_ticket; - C4::Context->set_userenv( - $session->param('number'), $session->param('id'), - $session->param('cardnumber'), $session->param('firstname'), - $session->param('surname'), $session->param('branch'), - $session->param('branchname'), $session->param('flags'), - $session->param('emailaddress'), $session->param('shibboleth'), - $session->param('desk_id'), $session->param('desk_name'), - $session->param('register_id'), $session->param('register_name') - ); + C4::Context->set_userenv_from_session($session); } # $return: 0 = invalid user @@ -1751,15 +1743,7 @@ sub check_api_auth { $session->param( 'interface', 'api' ); } $session->param( 'cas_ticket', $cas_ticket); - C4::Context->set_userenv( - $session->param('number'), $session->param('id'), - $session->param('cardnumber'), $session->param('firstname'), - $session->param('surname'), $session->param('branch'), - $session->param('branchname'), $session->param('flags'), - $session->param('emailaddress'), $session->param('shibboleth'), - $session->param('desk_id'), $session->param('desk_name'), - $session->param('register_id'), $session->param('register_name') - ); + C4::Context->set_userenv_from_session($session); return ( "ok", $cookie, $sessionID ); } else { return ( "failed", undef, undef ); @@ -1876,15 +1860,8 @@ sub check_cookie_auth { # No need to override the interface, most often set by get_template_and_user C4::Context->interface( $session->param('interface') ); } - C4::Context->set_userenv( - $session->param('number'), $session->param('id') // '', - $session->param('cardnumber'), $session->param('firstname'), - $session->param('surname'), $session->param('branch'), - $session->param('branchname'), $session->param('flags'), - $session->param('emailaddress'), $session->param('shibboleth'), - $session->param('desk_id'), $session->param('desk_name'), - $session->param('register_id'), $session->param('register_name') - ); + + C4::Context->set_userenv_from_session($session); if ( C4::Context->preference('TwoFactorAuthentication') ne 'disabled' ) { return ( "additional-auth-needed", $session ) if $session->param('waiting-for-2FA'); -- 2.34.1