Single Sign On (Shibboleth) + Bootstrap OPAC
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned | ||
3.10 |
Fix Released
|
Medium
|
Unassigned | ||
3.11 |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Bug 1871211 introduced the ability for patrons to login to the Evergreen OPAC using a Single Sign On service. This required changes to 2 tt2 files, which were only applied to the classic TPAC, not the Bootstrap OPAC.
We should apply those changes to the Bootstrap OPAC too! One wrinkle I noticed while poking around: the bootstrap OPAC has two different login form files which are very similar -- one for when the login form is in a modal, another for when it is not. Maybe those two files could be combined as part of this work, so we only need to add the SSO functionality to one login form, and we keep things more maintainable down the road. Those two login forms are at:
* Open-ILS/
* Open-ILS/
tags: | added: shibboleth sso |
tags: | added: pullrequest |
Changed in evergreen: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in evergreen: | |
assignee: | nobody → Jeff Davis (jdavis-sitka) |
Changed in evergreen: | |
milestone: | 3.11.1 → 3.12-beta |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
I've got a first pass at this here: /git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads / working/ collab/ jboyer/ lp1917083_ thats_sso_ bpac
https:/
Unfortunately the SSO bits don't actually work, you're just routed to the native login page (/eg/opac/login). If you disable native login you get a generic username or password was invalid message.
The actual login fields have been pulled out into a reusable chunk that's included by both the modal and regular login pages. I've also normalized the text around logging in (Login to Evergreen, Log in to My Account, Sign in with..., no, no, no.)
I'll keep poking at it but if anyone has time to see what I may be missing (I suspect either a problem with the when and where's of something in a .tt2 or something in EGCatLoader.pm) I've thrown it into a collab branch.