https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
Jonathan Druart <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] | |ommunity.org --- Comment #5 from Jonathan Druart <[email protected]> --- I am not going to test this patch, but I would like to make sure the changes make sense. 1. I do not understand the code in opac-user.pl, why is it here? It sounds like it will be better at its place in C4::Auth. 2. You should move out get_session the code you copy/pasted from there 3. I guess it is just my undestanding of CAS that is bad but why do we call get_session then delete it? Is cas_ticket == sessionID? Side note: it will not work if SessionStorage=memcached (but we should remove that), CGI::Session::Driver::memcached->traverse does nothing. Another one: from the pod of CGI::Session->find : "Experimental feature. Note: find() is meant to be convenient, not necessarily efficient. It's best suited in cron scripts." -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
