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

[readme] SAML2.0 does support passive refresh via isPassive #1187

Open
hauntingEcho opened this issue Feb 7, 2018 · 1 comment
Open

[readme] SAML2.0 does support passive refresh via isPassive #1187

hauntingEcho opened this issue Feb 7, 2018 · 1 comment

Comments

@hauntingEcho
Copy link

hauntingEcho commented Feb 7, 2018

In the readme, it is mentioned that "SAML doesn't provide a non-interactive way to refresh assertions". However, this is the purpose of the "isPassive" attribute on an AuthnRequest (line 2047 of SAMLCore)

@ericchiang
Copy link
Contributor

"isPassive" just seems to imply that the provider should actively re-log you in, kind of like OpenID Connect's "max_age" http://openid.net/specs/openid-connect-core-1_0.html#AuthRequest

This doesn't let us refresh the data, just implies that your provider is okay with not actively logging you back in (such as prompting your password).

For refresh tokens, dex expects to be able to refresh the claims in the id_token. If you've changed groups, those groups should be different in the new id_token you get from a refresh response. I don't see how "isPassive" achieves this.

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

No branches or pull requests

2 participants