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

--- Comment #22 from Tomás Cohen Arazi <[email protected]> ---
Created attachment 87545
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87545&action=edit
[UGLY FIX] Bug 18205: Set userenv in
Koha::REST::V1::Auth::authenticate_api_request

Koha code (like C4::Log::logaction) relies on C4::Context::userenv to be
set. This could be fixed in many ways. The easiest way is to set that
environmental variables after authorization happens.

I don't think this is a good design, but I'm not sure adding a new param
to logaction is the best option.

I'd rather make the action_logs table another target for Log4Perl at
some point. But I strongly disagree on making tiny enhancements be
required to refactor all the things :-D

I drop this code here for others to weight in.

Signed-off-by: Tomas Cohen Arazi <[email protected]>

-- 
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/

Reply via email to