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

id_token_hint not provided when logging out #720

Closed
bonnetb opened this issue Mar 7, 2022 · 7 comments
Closed

id_token_hint not provided when logging out #720

bonnetb opened this issue Mar 7, 2022 · 7 comments

Comments

@bonnetb
Copy link
Contributor

bonnetb commented Mar 7, 2022

Issue and Steps to Reproduce

Log-in, then log-out. Log-out will open OIDC Auth Server logout URL without passing id_token_hint as a URL parameter

Versions

4.2.9

Screenshots

Expected

id_token_hint in logout URL request

Actual

logout URL requested with no parameter

Additional Details

  • Installed packages:
@bonnetb
Copy link
Contributor Author

bonnetb commented Mar 7, 2022

https://github.com/AxaGuilDEv/react-oidc/blob/6c146e85a4ac492693095478991883623e406729/packages/context/src/oidc/vanilla/oidc.ts#L438 should be replaced with a more sophisticated call to logout URL

@guillaume-chervet
Copy link
Contributor

Thank you very much for your feedback. It is a must that have quicky to be done.

The issue duplicate #713

@guillaume-chervet
Copy link
Contributor

This Pull Request merge would be nice => openid/AppAuth-JS#198

@guillaume-chervet
Copy link
Contributor

I'am waiting for any news from them => openid/AppAuth-JS#209

@guillaume-chervet
Copy link
Contributor

I have the answer from the issue we will have to implment it here.

@guillaume-chervet
Copy link
Contributor

Hi @bonnetb , this is done in the lastest version 5.
Thank you very much for the issue. I close it feel free to reopen it if you need it.

@abhilashlr7
Copy link

This happens for us now. when we simply invoke logout() from useOidc() hook. Perhaps, the issue is not reproducible often, and has come back up post keycloak 19 version that's deployed on our server.

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

3 participants