Bug 13507

Summary: Add intranet support for CAS authentication
Product: Koha Reporter: Matthias Meusburger <matthias.meusburger>
Component: AuthenticationAssignee: Matthias Meusburger <matthias.meusburger>
Status: CLOSED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: enhancement    
Priority: P5 - low CC: dpavlin, jonathan.druart+koha, katrin.fischer, martin.renvoize, sandboxes, tmisilo, tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 19373    
Attachments: proposed patch
unit test
Bug 13507: Add intranet support for CAS authentication
Bug 13507 [Follow-up] : Add unit test.
Bug 13507: Add intranet support for CAS authentication
Bug 13507: Add intranet support for CAS authentication
Bug 13507: Add intranet support for CAS authentication
Bug 13507 [Follow-up] : Add unit test.
Bug 13507: Add intranet support for CAS authentication
[PASSED QA] Bug 13507 [Follow-up] : Add unit test.
[PASSED QA] Bug 13507: Add intranet support for CAS authentication
Bug 13507: Add intranet support for CAS authentication

Description Matthias Meusburger 2015-01-02 15:22:33 UTC
Add intranet support for CAS authentication.
Comment 1 Matthias Meusburger 2015-01-02 16:14:29 UTC Comment hidden (obsolete)
Comment 2 Matthias Meusburger 2015-01-02 16:14:53 UTC Comment hidden (obsolete)
Comment 3 Matthias Meusburger 2015-01-02 16:26:26 UTC
Test plan:

CAS Intranet setup is identical to OPAC setup, but you have to set the StaffClientBaseUrl syspref (with the http or https prefix / http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7770 for more infos on why it is needed)

Check that CAS login works in the intranet (except for the koha user, which is a special user).

Check that CAS still works at the OPAC.

Check that database login still works at both intranet and OPAC.

Check that authenticating against multiple CAS servers work (see C4/Auth_cas_servers.yaml.orig)
Comment 4 Matthias Meusburger 2015-03-02 14:22:58 UTC Comment hidden (obsolete)
Comment 5 Matthias Meusburger 2015-03-02 14:23:10 UTC Comment hidden (obsolete)
Comment 6 Matthias Meusburger 2015-03-02 15:03:54 UTC Comment hidden (obsolete)
Comment 7 Matthias Meusburger 2015-03-02 15:59:27 UTC Comment hidden (obsolete)
Comment 8 Matthias Meusburger 2015-03-02 16:03:36 UTC Comment hidden (obsolete)
Comment 9 sandboxes@biblibre.com 2015-03-02 16:45:14 UTC
Patch tested with a sandbox, by Koha Team AMU <koha.aixmarseille@gmail.com>
Comment 10 sandboxes@biblibre.com 2015-03-02 16:45:41 UTC Comment hidden (obsolete)
Comment 11 sandboxes@biblibre.com 2015-03-02 16:45:44 UTC Comment hidden (obsolete)
Comment 12 Katrin Fischer 2015-03-06 10:38:45 UTC
Created attachment 36671 [details] [review]
[PASSED QA] Bug 13507 [Follow-up] : Add unit test.

Signed-off-by: Koha Team AMU <koha.aixmarseille@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 13 Katrin Fischer 2015-03-06 10:39:46 UTC Comment hidden (obsolete)
Comment 14 Katrin Fischer 2015-03-06 10:41:57 UTC
Martin, could you take a final look at the code for this? 

In my testing it worked alright, tested with a biblibre test CAS server - Matthias can give you the data for that. (Very useful!)
Comment 15 Tomás Cohen Arazi 2015-03-30 16:33:57 UTC
Guys, this doesn't apply, could you please fix the conflict? I'm willing to push it ASAP
Comment 16 Matthias Meusburger 2015-04-02 10:23:12 UTC
Created attachment 37431 [details] [review]
Bug 13507: Add intranet support for CAS authentication

This patch allows to use CAS authentication for intranet login.

 It works exactly the same as the OPAC login, except that the
 staffClientBaseURL syspref must be set for intranet login
 (like OPACBaseURL must be set for OPAC login).

Signed-off-by: Koha Team AMU <koha.aixmarseille@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 17 Matthias Meusburger 2015-04-02 10:23:43 UTC
It was just a template issue, rebased.
Comment 18 Tomás Cohen Arazi 2015-04-08 15:22:09 UTC
Feature pushed to master.

Thanks Matthias and congrats!