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 support for SSO logout #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

axelvnk
Copy link

@axelvnk axelvnk commented Jan 23, 2018

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.0%) to 96.121% when pulling f5358c6 on Inuits:sso_logout into c32db2e on lightSAML:master.

@axelvnk
Copy link
Author

axelvnk commented Jan 23, 2018

@tmilos the php-cs-fixer errors are on code that already existed in the master branch. Should I open up a new pull request just for this? Because its unrelated to this PR.

@cfoehrdes
Copy link

Is there a chance that this feature and the corresponding lightSAML/lightSAML#106 will be merged?

@axelvnk
Copy link
Author

axelvnk commented May 22, 2018

@cfoehrdes probably not since they got this https://github.com/lightSAML/lightSAML-logout

@cfoehrdes
Copy link

@axelvnk oh ok, thanks for the hint to this project. I will try to integrate this into our symfony application. Have you tried the lib?

@axelvnk
Copy link
Author

axelvnk commented May 22, 2018

No i made my own integration before i spotted their extra lib... It's whats in this PR

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

Successfully merging this pull request may close these issues.

3 participants