-
Notifications
You must be signed in to change notification settings - Fork 45
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
Consider using OneLogin Simple SAML toolkit for PHP #60
Comments
Last year @davidstrauss mentioned positively it in Power Users: https://groups.google.com/a/pantheon.io/d/msg/power-users/uBHYj1c8lqY/PsIbFvZHEgAJ From what I see it looks like a much nicer option. And between LCache and Solr Power we have better conventions for handling a Composer dependency like OneLogin's library than what is currently recommended here with SimpleSAML. |
+1 for this. There was a time when I thought that SimpleSamlPHP support was important, and retaining it as an option could be nice. HOWEVER - if we can do SAML out of the box with OneLogin's library, operating as a Service Provider only, and with a single configuration file to set up, that would be... preferable. Worth a spike at least. |
@danielbachhuber what's the status of this? Just integrated a site with this plugin and SimpleSamlPHP. No rush, just want to put it into the timeline to switch @stevector how do you currently handle this? I opted with a post composer hook to |
@schrapel Functional work in progress on #68. Remaining work is:
|
@schrapel I haven't used the onelogin library myself yet.
I assume you are using the |
I was talking about simplesamlphp. You have to store the config files in the simplesaml folder unless you define a nginx/apache env variable which I wasn't sure if they were possible on pantheon. So I chose to copy these files into the vendor folder after composer install.
And yes using the web folder
|
Ah, right. |
🚢 |
OneLogin has a Simple SAML toolkit for PHP, which looks robust. We may be able to package this directly with the plugin, instead of requiring SimpleSAMLphp to be installed separately.
The text was updated successfully, but these errors were encountered: