-
Notifications
You must be signed in to change notification settings - Fork 7
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 defaultAuthenticationMethod #11
Comments
If there is a default supplied by the IdP and the RP sends an explicit contextClassRef in the request then we need to honor the explicit request by the SP rather than do what the IdP has specified by the default. |
Thinking about this a little more, there are two scenarios affecting this RFE 1 SP does the wrong thing & IdP wants to enforce policy. Use CaseIdP wants to force 2FA for a SP, but the SP insists on sending PasswordProtectedTransport. Due to clueless vendor the vendor can't / won't change the AuthnContextClassRef requested in the AuthRequest. Therefore IdP would want the relying-party.xml default to override. 2 SP is well-behaved and actually wants to offer 2 levels of authN Use CaseIdP default is PasswordProtectedTransport and SP sends unspecified. When the user tries to do something requiring a better auth level (like change their direct-deposit) then the SP will send a new AuthRequest with 2FA, and needs the IdP to respect/perform that. |
I've been uncomfortable about the first use case, but I guess we allow Do we need to do anything to make the second use case work? Doesn't it David On 07/19/2014 05:40 PM, David Langenberg wrote:
|
Fixed in version 1.2.1 |
Shibboleth relying party configuration allows a default authentication method to be specified per relying party. Make use of the default value when a RP does not send a method (context) with their request.
Still outstanding is whether to use the default method when the RP does send a value. If so, does it get added to the beginning or the end of the list?
The text was updated successfully, but these errors were encountered: