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

Adds option to use a Shibboleth SP discovery feed for metadata filtering #53

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

klelifo
Copy link

@klelifo klelifo commented May 10, 2016

Hi!
First of all: Great work on this tool and thanks for all the work you put into it!

And now: I did exactly what I was told not to do, cloned the repository and hosted my own DiscoJuice instance :) (at least in parts)

With this pull-request I added support to filter the great number of IdPs coming from one or more of the involved federations down to ones that are actually granted access to a certain Shibboleth SP.

As far as I understand it, the Shibboleth SP provides a special discovery feed URL that gives you a list of all IdPs that are known to the SP.
With this list I filtered the federation metadata, giving me a nice small list of IdPs that are allowed to use my SP.

Example is up and running at https://fauboxtest.rrze.uni-erlangen.de
Click on "SSO Anmeldung" to see the DiscoJuice overlay with the filtered list of IdPs.
See https://fauboxtest.rrze.uni-erlangen.de/Shibboleth.sso/DiscoFeed for the generated Shibboleth SP discovery feed.

I hope this is helpful to you.
Thx and cu

@klelifo
Copy link
Author

klelifo commented May 13, 2016

I also added a fix for #42

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.

1 participant