Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added uc_uid mapping to shibboleth authentication #655

Merged
merged 5 commits into from
Aug 7, 2014

Conversation

thecoolestguy
Copy link
Member

Some institutions require comparison to an authentication field other than email address. I've added the institution id (uc_uid) field as an option as it is the probably the next most in-demand to the email address.

@jrjohnson
Copy link
Member

I'm ok with this if it fixes the problem right now, however my understanding of shibboleth is that the eppn (Edu personal principal something) is supposed to be the primary unique identifier for authentication. I think if we're going to add the option to pick your auth method it should include that attribute.

@jrjohnson
Copy link
Member

Never mind - I see that if I set it to uc_uid and store the eppn there I can then set the ilios_authentication_shibboleth_user_id_attribute to eppn and that will do it.

jrjohnson added a commit that referenced this pull request Aug 7, 2014
added uc_uid mapping to shibboleth authentication
@jrjohnson jrjohnson merged commit bac2981 into ilios:master Aug 7, 2014
@thecoolestguy thecoolestguy deleted the uci_custom_shib branch May 24, 2016 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants