[Koha-bugs] [Bug 17427] Replace CGI::Session with Data::Session

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Jul 5 10:04:32 CEST 2023


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17427

--- Comment #102 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
Created attachment 153038
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153038&action=edit
Bug 17427: HACKY - force new session

Look at Data::Session->new, it's calling $self -> load_session;
Then look at load_session then user_id
=> we are retrieving the ID from the cookie

But in our Auth.t tests we are mocking CGI->cookie to always return the
previous CGISESSID

We should certainly adjust the test here, and remove this patch.

194         # Note: We can test return values from checkauth here since we
mocked the safe_exit after the Redirect 303
195         is( $return[0], $patron2->userid, 'Login of patron2 approved' );
196         isnt( $return[2], $sessionID, 'Did not return previous session ID'
);
197         ok( $return[2], 'New session ID not empty' );

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list