Bug 28571

Summary: C4::Auth::_session_log is not used and must be removed
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: dcook, fridolin.somers, kyle, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.11.00,21.05.02
Attachments: Bug 28571: Remove C4::Auth::_session_log
Bug 28571: Remove C4::Auth::_session_log
Bug 28571: Remove C4::Auth::_session_log

Description Jonathan Druart 2021-06-16 13:27:39 UTC
it's logging in a /tmp file
Comment 1 Jonathan Druart 2021-06-16 13:28:49 UTC
Created attachment 122039 [details] [review]
Bug 28571: Remove C4::Auth::_session_log

It's not used and must be removed

Test plan:
  % git grep _session_log
must not return any result.
Comment 2 David Nind 2021-06-18 19:29:47 UTC
Created attachment 122173 [details] [review]
Bug 28571: Remove C4::Auth::_session_log

It's not used and must be removed

Test plan:
  % git grep _session_log
must not return any result.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Marcel de Rooy 2021-06-23 14:34:28 UTC
Created attachment 122342 [details] [review]
Bug 28571: Remove C4::Auth::_session_log

It's not used and must be removed

Test plan:
  % git grep _session_log
must not return any result.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Jonathan Druart 2021-06-24 09:54:19 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 5 Kyle M Hall 2021-07-09 14:41:25 UTC
Pushed to 21.05.x for 21.05.02
Comment 6 Fridolin Somers 2021-08-02 20:36:30 UTC
I think we should not impact olders versions