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

POST binding support for authnrequest #422

Open
wants to merge 5 commits into
base: 3.4.1
Choose a base branch
from

Conversation

Fredthelead
Copy link

Added POST binding support for the SingleSignOnService endpoint

@pitbulk
Copy link
Contributor

pitbulk commented Jan 22, 2020

Is not that easy, what if you need to sign the AuthNrequest?

I don't gonna merge this since add complexity to the toolkit, but will leave the PR open in case someone needs this feature.

@teuf22
Copy link

teuf22 commented Mar 16, 2020

We also require POST binding. Would be very nice if this is merged in base branch!

@Fredthelead
Copy link
Author

@pitbulk FYI I've now added support for signed AuthNRequests over POST binding 🙂

@joonlabs
Copy link

@pitbulk It would be so great to be able to use this in the package itself. Is there any way to use POST binding with signed AuthNRequests? We would really need this, since there are IDPs who only support POST binding.

@Fredrik82 if there is no official way, what would be the easiest way to integrate this?

@jaulz
Copy link

jaulz commented Dec 3, 2024

Did you actually find a workaround how to achieve this?

@joonlabs
Copy link

joonlabs commented Dec 3, 2024

Did you actually find a workaround how to achieve this?

Yes. We implemented the endpoints by our own. Similar to how the pull request handles the logic.
But it looks like we are going to ditch this library in the near future, because the development of it is really slow - e.g. we have a pull request that is waiting for a release for months now :(

@jaulz
Copy link

jaulz commented Dec 3, 2024

@joonlabs do you know any better alternative?

@joonlabs
Copy link

@jaulz not really yet. I am in contact with the guys behind https://github.com/simplesamlphp/saml2 - it looks promising but is not ready yet to be used as a replacement for my understanding

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.

5 participants