Bug 20730

Summary: Missing authentication check in serials/routing.pl
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, katrin.fischer, nick
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 20351    
Attachments: Bug 20730: Move the authentication block before doing anything
Bug 20730: Move the authentication block before doing anything
Bug 20730: Move the authentication block before doing anything

Description Jonathan Druart 2018-05-08 17:21:21 UTC

    
Comment 1 Jonathan Druart 2018-05-08 17:28:25 UTC
Created attachment 75177 [details] [review]
Bug 20730: Move the authentication block before doing anything

No need to add more information here, it's self-explanatory...
Comment 2 Mark Tompsett 2018-05-08 17:49:17 UTC
Created attachment 75179 [details] [review]
Bug 20730: Move the authentication block before doing anything

No need to add more information here, it's self-explanatory...

NOTE: trivial reformat and reposition,
      with only pointless debug => 1 line removed.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 3 Tomás Cohen Arazi 2018-05-08 18:00:19 UTC
Created attachment 75181 [details] [review]
Bug 20730: Move the authentication block before doing anything

No need to add more information here, it's self-explanatory...

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Jonathan Druart 2018-05-24 16:44:18 UTC
Patch pushed to master for 18.05


Also to 17.11.06 and 17.05.12