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

Add scoping option to AuthnRequest #214

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

Conversation

Ricky-rick
Copy link

@Ricky-rick Ricky-rick commented May 5, 2017

Add scoping to a service provider to specify a list of identity providers in an authnRequest to a proxying identity provider.
Fix Issue #211

Travis fails on pdepend wich has no relation with this PR
PHP Fatal error: Interface 'PHP_CodeSniffer_Sniff' not found in /home/travis/build/onelogin/php-saml/tests/ZendModStandard/Sniffs/Debug/CodeAnalyzerSniff.php on line 30

@Ricky-rick
Copy link
Author

I updated the pull request so it has no conflicts with the master branch.
Can you merge the pr or give me a hint on how to solve the issue?

@stayallive
Copy link

I would go for maintaining a fork with this patched in since I don't think it will be accepted.

Reference: #137

@arnoudhgz
Copy link

I would really like this feature.
It does not really make it more difficult for users to use this package. If they don't know the feature, they don't care.

@thierrymaasdam
Copy link

I was just wondering, being in need of this feature atm, if there’d be any chance for it to be added to the package officially.

Looking forward to a reply. Thank you in advance!

@pitbulk
Copy link
Contributor

pitbulk commented Nov 30, 2018

Not official support for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants