-
Notifications
You must be signed in to change notification settings - Fork 94
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
Several identity providers #94
Comments
Yes that would be interesting. In todays workloads there are often multiple IDPs and it's also needed to configure them dynamically while application lifecycle because you might manage them in your database or similar. |
As we urgently needed that, I just built it. See my PR. |
[Repost from PR just in case ;)] Hi guys, I'm just seeing this issue and this is great news for us because we are looking to integrate SAMLv2 login in our application, for various distinct customers, and thus we need multiple idps. (see my original stackoverflow question here). We initially were planning to create our own open source bundle (another one...) to address this issue since no other one seemed to be interested by this feature. I think it should be better for the Symfony community to minimise the number of similar bundles out there and we'd be interested to contribute here and help implementing this feature on this bundle. How could we help there? How should we plan to support this while do not making BC breaks (or bumping major version with BC breaks inside?). We'd be glad to support this change and make a PR once we're okay on the way to implement/configure it. Best |
The PR was merged but it introduced big BC breaks and other issues. It seems this features needs some BC break and it should be tested properly. |
Great to read that. What is your definition of "soon" ? :) We'll need it by the end of the year, and will fork your repo and contribute. If we could help you on that feature, please don't hesitate to tell us. Do you want us to test it? It would be great if we could help you the right way on that matter :) Cheers 🍻 |
It would be great if you can test it with Symfony 3 and 4 and give some feedback. Unit tests should be updated too, that will help a lot. |
We are using it in a production application with Symfony 4.3.8 and API Platform. |
I would love this feature too. Thanks |
You can implement that yourself using CompilerPass in Symfony. |
Can anyone tell me if support for multiple IDPs was ever added officially? I can see the experimental multipleidps branch is a couple of years old now. |
@tobyski-tdsultra, multiple IdP support not added yet. |
It is a mandatory evolution for me. I am forced to use another solution. My application must establish an sso connection to 2 different IdPs. |
If you using Symfony 6 for your application, you can use nbgrp/onelogin-saml-bundle that supports multiple IdP configuration. |
This is great news, but what if we are not able yet to use Symfony 6? Any advice? |
Unfortunately I have no advice right now. |
Hello there, We are upgrading an old SF 3.4 project to newer SF5, and the old SAML bundle I used is no longer maintained for SF5. I am wishing to use this bundle, but we must have different IDPs configured for our project. I cannot use the SF6 https://github.com/nbgrp/onelogin-saml-bundle because SF6 breaks several other bundles we are using. |
@gprince64 hi. |
Hi @hslavich!
Is it possible to have several different identity providers?
For example:
config.yml
hslavich_onelogin_saml:
idp1:
........
idp2:
........
sp:
.......
How can I dynamically modify config if not?
The text was updated successfully, but these errors were encountered: