Bug 23026

Summary: Staff client CAS URL escaped twice, preventing login
Product: Koha Reporter: Nick Clemens (kidclamp) <nick>
Component: AuthenticationAssignee: Nick Clemens (kidclamp) <nick>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: dpavlin, fridolin.somers, jonathan.druart, lucas, martin.renvoize
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:
Circulation function:
Attachments: Bug 23026: Do not double encode CAS url

Description Nick Clemens (kidclamp) 2019-05-31 16:35:19 UTC
Bug 21973 fixed CAS login for the opac - we need to do the same for the staff client
Comment 1 Nick Clemens (kidclamp) 2019-05-31 16:38:01 UTC
Created attachment 90235 [details] [review]
Bug 23026: Do not double encode CAS url

It is already escaped correctly in
C4::Auth_with_cas::_url_with_get_params using URI::Escape::uri_escape

Note that shibbolethLoginUrl is not and must be url escaped in template
("be consistent, they said")
Comment 2 Jonathan Druart 2019-05-31 17:51:56 UTC

*** This bug has been marked as a duplicate of bug 22585 ***