-
Notifications
You must be signed in to change notification settings - Fork 210
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
SAML with Shibboleth support #55
Comments
+1 |
@Nogbit I would like to see SAML support as well. If someone is willing to contribute it then that would be the fastest way to get it into bell. |
We decided to stick with Express for the time being, if one of us gets the bandwidth we could contribute and we did discuss that. |
thinking if you take passport-saml and create the wrapper for hapi-bell it might work. Ideally, with maintaining passport-saml saml.js as a library, not forking. Anyone interested ? More |
+1 also interested in this |
+1 non-SAML support is making us stick with Express for now, but would love to help and contribute. |
I'd love to be able to use SAML with bell. Any thought on where to start? |
You would have to implement it. So you would add a new protocol to bell called SAML. A protocol is really this:
You can find the complicated specs here https://wiki.oasis-open.org/security/FrontPage#SAML_V2.0_Standard and there https://docs.oasis-open.org/security/saml/v2.0/saml-core-2.0-os.pdf If you do decide to try writing it, you may make a PR request and we can see how we can help you to get it done properly. |
I managed to use passport-saml with hapi. I think that it should be a separate plugin as it has a specific auth flow than OAuth (e.g. metadata endpoint). I probly work on this later this year, but suffice to say that integrating it to bell is not straightforward |
Thanks @molekilla for giving this is a spin! Would you care to share how you integrated passport-saml into hapi? |
here you go |
Closing due to inactivity. |
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
It seems this was supported in travelogue with passport. I still don't know why travelogue was abandoned in favor of bell but it kind of leaves new projects in no mans land that that need SAML support.
The text was updated successfully, but these errors were encountered: