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

Shibboleth: update shibboleth2.xml entityID to end with "sp" rather than "shibboleth" #2104

Closed
pdurbin opened this issue Apr 27, 2015 · 8 comments
Assignees
Labels
Milestone

Comments

@pdurbin
Copy link
Member

pdurbin commented Apr 27, 2015

Per https://spaces.internet2.edu/display/InCFederation/Entity+IDs it has been recommended to use to have our Shibboleth entityID be in this form...

entityID="https://sp_name.example.edu/sp"

... rather than this one:

entityID="https://sp_name.example.edu/shibboleth"

We should update the example at https://github.com/IQSS/dataverse/blob/master/conf/vagrant/etc/shibboleth/shibboleth2.xml

@pdurbin pdurbin added the Type: Suggestion an idea label Apr 27, 2015
@scolapasta scolapasta added this to the Candidates for 4.0.1 milestone May 8, 2015
@eaquigley eaquigley modified the milestones: Candidates for 4.0.1, In Design May 8, 2015
@pdurbin pdurbin changed the title Shibboleth: update sample shibboleth2.xml confige entityID to https://sp_name.example.edu/sp Shibboleth: update sample shibboleth2.xml config entityID to https://sp_name.example.edu/sp Jul 20, 2015
@pdurbin
Copy link
Member Author

pdurbin commented Jul 20, 2015

We should update the example at https://github.com/IQSS/dataverse/blob/master/conf/vagrant/etc/shibboleth/shibboleth2.xml

More importantly, we've already updated the example at http://guides.dataverse.org/en/latest/installation/shibboleth.html to be entityID="https://shibtest.dataverse.org/sp".

However, all of our servers still have the old style ending in "shibboleth" rather than "sp". For example, https://dataverse-demo.iq.harvard.edu/Shibboleth.sso/Metadata shows entityID="https://dataverse-demo.iq.harvard.edu/shibboleth".

@scolapasta and @kcondon I'm thinking we could use this issue to work on making sure all of our servers use the new ending. Also, we should re-define which servers we want Shibboleth support on. Here's a potential list:

In INC01226245 we're working on getting https://dataverse.harvard.edu registered with InCommon at https://incommon.org/federation/info/all-entity-categories.html#SPs but we could probably have a few testing or staging instances. In the past we've done "out of band" metadata exchanges for Harvard's Identity Provider (IdP) but since the Harvard IdP is part of InCommon we should probably treat it like all other IdPs in that federation.

@mercecrosas mercecrosas modified the milestones: In Design, In Review Nov 30, 2015
@pdurbin pdurbin removed their assignment Jan 21, 2016
@bencomp
Copy link
Contributor

bencomp commented Jan 22, 2016

The Entity IDs doc also has a warning that says

Do NOT change your entity ID!

So in any case it's important to choose wisely before registering and using an ID. Add to #2884?

@scolapasta scolapasta removed this from the Not Assigned to a Release milestone Jan 28, 2016
@pdurbin
Copy link
Member Author

pdurbin commented Feb 4, 2016

https://dataverse-demo.iq.harvard.edu was renamed to https://demo.dataverse.org in #2651.

To me this issue is about registering the following two servers at https://incommon.org/federation/info/all-entities.html#SPs

As of this writing https://dataverse.harvard.edu is using a non-recommended entityID (doesn't end in "sp") and https://demo.dataverse.org isn't configured for Shibboleth at all (after the rename). To make progress on this I could at least reconfigure demo with a recommended entityID (I may want to change the docs from shibtest to demo) and get it working again with http://www.testshib.org .

Incidentally, I noticed that many QA/testing/staging/demo Service Providers are registered with InCommon...

incommon_federation_info_entities_-_2016-02-04_09 55 26
incommon_federation_info_entities_-_2016-02-04_09 56 00
incommon_federation_info_entities_-_2016-02-04_09 56 38

... and I think one server should be enough but I'll check with @kcondon if he'd like more from the list at #2104 (comment) registered with InCommon.

@pdurbin pdurbin self-assigned this Feb 4, 2016
@pdurbin
Copy link
Member Author

pdurbin commented Feb 4, 2016

I checked in with @kcondon and he said that ideally the following servers would be registered with InCommon:

I'm adding public vs. private above because I think that all servers would necessarily become public as part of the InCommon registration process. This means that they would need to be given stable hostnames and have real SSL certs, etc.

I don't think we should bother registering https://apitest.dataverse.org and in general I think there's agreement that this should some day be deprecated in favor of https://demo.dataverse.org

Likewise, it's probably not worth registering http://phoenix.dataverse.org . It's actively used but we don't need to clutter up the InCommon list and there seem to be enough other hosts to test on. Same with cluster/staging which is used for load testing.

I'm planning on using https://shibtest.dataverse.org for testing in the near term but I don't think it needs to be registered with InCommon either. We can use beta or some other server.

@pdurbin
Copy link
Member Author

pdurbin commented Feb 4, 2016

I just changed the entityID on https://demo.dataverse.org to the one we want to use (entityID="https://demo.dataverse.org/sp"). I changed it in two places:

[root@dataverse-demo ~]# cd /etc/shibboleth
[root@dataverse-demo shibboleth]# diff shibboleth2.xml.2016-02-04.oldhostname shibboleth2.xml
16c16
<     <ApplicationDefaults entityID="https://dataverse-demo.iq.harvard.edu/shibboleth"
---
>     <ApplicationDefaults entityID="https://demo.dataverse.org/sp"
31c31
<             <SSO discoveryProtocol="SAMLDS" discoveryURL="https://dataverse-demo.iq.harvard.edu/loginpage.xhtml">SAML2 SAML1</SSO>
---
>             <SSO discoveryProtocol="SAMLDS" discoveryURL="https://demo.dataverse.org/loginpage.xhtml">SAML2 SAML1</SSO>
[root@dataverse-demo shibboleth]# 

Running service shibd restart was enough to download the new metadata from https://demo.dataverse.org/Shibboleth.sso/Metadata but insufficient for login to work properly. I had to run service httpd restart as well. Now it works again for TestShib users (I re-uploaded the metadata per http://guides.dataverse.org/en/4.2.3/installation/shibboleth.html#testing ) but not for Harvard or MIT users. They'll see errors like below until we either join InCommon or ask for another out-of-band metadata exchange:

harvardsp
mitsp

2016-03-07 update: The TestShib metadata expired again, resulting in the usual "Something horrible happened... Error Message: SAML 2 SSO profile is not configured for relying party" error:

testshib-horrible

Rather than re-uploading our metadata I configured https://demo.dataverse.org for InCommon (as I mentioned at #2937) even though we haven't been registered with them.

@pdurbin
Copy link
Member Author

pdurbin commented Feb 25, 2016

As of yesterday (INC01606454) https://dataverse.harvard.edu/Shibboleth.sso/Metadata and https://dataverse.harvard.edu/Shibboleth.sso/DiscoFeed are back in action. I chatted with @bencomp about this at http://irclog.iq.harvard.edu/dataverse/2016-02-25

We still need to (via INC01226245):

Once these are registered with InCommon I'll consider this issue ready for QA.

@pdurbin pdurbin changed the title Shibboleth: update sample shibboleth2.xml config entityID to https://sp_name.example.edu/sp Shibboleth: update shibboleth2.xml entityID and register with InCommon Mar 3, 2016
@pdurbin pdurbin changed the title Shibboleth: update shibboleth2.xml entityID and register with InCommon Shibboleth: register with InCommon after updating entityID Mar 8, 2016
@pdurbin
Copy link
Member Author

pdurbin commented Mar 21, 2016

A week ago the people working on INC01226245 replied and said they are actively trying to resolve it.

@pdurbin pdurbin changed the title Shibboleth: register with InCommon after updating entityID Shibboleth: update shibboleth2.xml entityID to end with "sp" rather than "shibboleth" Apr 22, 2016
@pdurbin
Copy link
Member Author

pdurbin commented Apr 22, 2016

@kcondon and I discussed how #2937 is really a better issue to leave open for getting Harvard servers registered with InCommon in INC01226245 since it's about federated login so I'm passing this to QA to close. I changed the title of this issue to be about changing the entityID to end in "sp" rather than "shibboleth" which we have done on all of our servers and in the guides.

@pdurbin pdurbin assigned kcondon and unassigned pdurbin Apr 22, 2016
@kcondon kcondon closed this as completed Apr 27, 2016
@pdurbin pdurbin added this to the 4.4 milestone Jun 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants