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

Tested and working idp's #294

Closed
sp90 opened this issue Aug 1, 2018 · 4 comments
Closed

Tested and working idp's #294

sp90 opened this issue Aug 1, 2018 · 4 comments

Comments

@sp90
Copy link

sp90 commented Aug 1, 2018

So the documentation states:

Passport-SAML has been tested to work with Onelogin, Okta, Shibboleth, SimpleSAMLphp based Identity Providers, and with Active Directory Federation Services.

But i wanted to start a tread where people could give feedback if they found it working with these and other not noted services

Remember to check if its been stated already, then if you have it working give it a 👍 and if you have it not working please use the 👎 instead, that way we keep the issue short and tidy

@sp90
Copy link
Author

sp90 commented Aug 1, 2018

Onelogin is working 👍

@markstos
Copy link
Contributor

markstos commented Aug 3, 2018

I've tested it with Onelogin, Okta, ADFS and many Shibboleth servers.

This module is generally spec-compliant, so it should work with spec-compliant servers. However, it has some missing features. The issue I've run into is not so much with different providers, it's when the idP wants to use some feature that's missing or not well supported here. So far I've got about a dozen or so integrations working with it. The one missing feature I needed was better support for multiple SAML providers configured at once, and I patched the module to improve that support.

The existing issues posted in this issue tracker provide a good sense of the places where people have run into rough edges.

@sp90
Copy link
Author

sp90 commented Aug 6, 2018

I'm using this branch to dynamically handle configs:
#276

@markstos
Copy link
Contributor

I'm closing this. Just because it works for one person with their IdP in one configuration isn't a guarantee it will work for you with your configuration. A more useful reference could be a list of SAML spec features that we support, or a list of SAML features that are known to be missing.

Also, we have a wiki for collaborative documentation.

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

No branches or pull requests

2 participants