Skip to content

Update OIDC Logout to work with expired id_tokens#1592

Merged
zachmargolis merged 1 commit intomasterfrom
margolis-handle-expired-id-token-hint
Aug 8, 2017
Merged

Update OIDC Logout to work with expired id_tokens#1592
zachmargolis merged 1 commit intomasterfrom
margolis-handle-expired-id-token-hint

Conversation

@zachmargolis
Copy link
Contributor

@zachmargolis zachmargolis commented Aug 7, 2017

Why:
To provide a smooth experience when trying to log out. The spec
states that:

Previously issued ID Token passed to the logout endpoint
as a hint about the End-User's current authenticated session
with the Client.

which we can interpret as "allows expired tokens" as well.

**Why**:
To provide a smooth experience when trying to log out. The spec
states that:
> Previously issued ID Token passed to the logout endpoint
> as a hint about the End-User's current authenticated session
> with the Client.
which we can interpret as "allows expired tokens" as well.
@zachmargolis zachmargolis self-assigned this Aug 7, 2017
Copy link
Contributor

@monfresh monfresh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zachmargolis zachmargolis merged commit 8be8a54 into master Aug 8, 2017
@zachmargolis zachmargolis deleted the margolis-handle-expired-id-token-hint branch August 8, 2017 22:40
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.

2 participants