Skip to content

Adds ability to validate Logout token#251

Open
like-a-bause wants to merge 5 commits into
coreos:v3from
like-a-bause:logoutToken
Open

Adds ability to validate Logout token#251
like-a-bause wants to merge 5 commits into
coreos:v3from
like-a-bause:logoutToken

Conversation

@like-a-bause
Copy link
Copy Markdown

Adds the ability to validate a Logout Token according to https://openid.net/specs/openid-connect-backchannel-1_0.html

ericchiang:mod branch is already merged.

Closes #211

@archerbj
Copy link
Copy Markdown

archerbj commented Jan 26, 2021

Sorry to bother, may I ask when this will be merged?

@flo-mic
Copy link
Copy Markdown

flo-mic commented Feb 14, 2022

@aspeteRakete do you have some insights why this PR is not merged?

@like-a-bause
Copy link
Copy Markdown
Author

I think @ericchiang was concerned that backchannel logout is still in draft state. As can be seen in this comment: #211 (comment)

The last rev of the spec was published on August 7, 2020. But at least it is not listed as inactive.
For reference:
https://openid.net/specs/openid-connect-backchannel-1_0.html
https://openid.net/developers/specs/

@flo-mic
Copy link
Copy Markdown

flo-mic commented Feb 14, 2022

Ah I see. Unfortunately it is two years ago since last update ☹️ and backchannel logout would be a huge security improvement from my perspective. Hope thy will finalize this soon.

Anyway, thanks for the PR 👍🏼. Would like to see it implemented

@like-a-bause
Copy link
Copy Markdown
Author

You can of course use my fork if you want to use backchannel logout. Just updated the branch to contain the latest upstream changes.

@lus
Copy link
Copy Markdown

lus commented Mar 27, 2022

Is there any update on this?
As far as I am aware, this would not be a breaking change and would allow us to implement backchannel logout without having to rely on an unofficial fork.
Backchannel logout is widely supported by several IdPs, Keycloak would be a popular example.

@lwj5
Copy link
Copy Markdown

lwj5 commented Sep 30, 2022

Is this PR moving forward? Looks like it's official https://openid.net/specs/openid-connect-backchannel-1_0.html
@ericchiang

@shkarface
Copy link
Copy Markdown

Since backchannel logout is official, I think this PR should be merged.

@FAUSheppy
Copy link
Copy Markdown

Any news here? Would greatly appreciate this critical feature.

@ruoibmt
Copy link
Copy Markdown

ruoibmt commented Jun 24, 2024

This is still desired

@marjuscako
Copy link
Copy Markdown

Hey @ericchiang, since this is official https://openid.net/specs/openid-connect-backchannel-1_0.html, could you take a look on this PR? This blocks an important feature for oauth2-proxy oauth2-proxy/oauth2-proxy#1224

@knadh
Copy link
Copy Markdown

knadh commented Feb 16, 2025

+1

@knadh
Copy link
Copy Markdown

knadh commented May 24, 2025

@ericchiang would you be able to provide an update on whether this PR is being considered? Thank you.

@Brenton-Keats
Copy link
Copy Markdown

I'm also very interested in this PR being completed. @ericchiang do you have an update on this? Thanks.

@ski7777
Copy link
Copy Markdown

ski7777 commented Oct 13, 2025

@mikedanese @rithujohn191
Can you push this topic? I think this is really important for many people.

@maciej-lech
Copy link
Copy Markdown

+1

2 similar comments
@bpitet-hw
Copy link
Copy Markdown

+1

@tiran133
Copy link
Copy Markdown

tiran133 commented May 6, 2026

+1

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.

Logout Token Validation Support?