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.

Sign up for GitHub

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

Jump to bottom

Support for ForceAuthn AuthnRequest #92

Closed
graaff opened this issue Jun 6, 2013 · 5 comments
Closed

Support for ForceAuthn AuthnRequest #92

graaff opened this issue Jun 6, 2013 · 5 comments

Comments

@graaff
Copy link
Contributor

graaff commented Jun 6, 2013

As far as I can tell it currently isn't possible to set ForceAuthn=true for an AuthnRequest, as I only see a mention of ForceAuthn in the saml20protocol_schema.xsd.

I guess this could be implemented in a manner similar to IsPassive in 8a825f3 ?
@sebastienluquet sebastienluquet mentioned this issue Jun 18, 2013
Closed
@graaff
Copy link
Contributor Author

graaff commented Jul 10, 2013

I might have a client who is willing to pay me to implement this, but I'm hesitant to go forward with this until I know that a) I'm on the right track here, and b) I can be reasonably sure a pull request will be accepted (barring quality issues, obviously). Could you please provide some feedback on this?

@stouset
Copy link
Contributor

stouset commented Jul 11, 2013

The same approach looks like it would be reasonable. Pull requests will gladly be accepted. :)

Ideally, we'd love to rework this entire gem to be much more standards-compliant, and feature-complete at least for the protocol methods that we support.

@graaff
Copy link
Contributor Author

graaff commented Aug 15, 2013

Unfortunately the client has decided to solve this the problem they were having in a different way, so they won't sponsor this functionality through us. :-(

@sebastienluquet sebastienluquet mentioned this issue May 28, 2014
Merged
@pitbulk
Copy link
Collaborator

pitbulk commented Sep 9, 2014

Related to #143

@pitbulk
Copy link
Collaborator

pitbulk commented Oct 31, 2014

This functionality was added.

@pitbulk pitbulk closed this as completed Oct 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@stouset @graaff @pitbulk