-
Notifications
You must be signed in to change notification settings - Fork 226
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
Silent Refresh with multiple tabs #967
Comments
If you are using Refresh Token rotation then I'm afraid this is a known issue with Keycloak: (more details here: keycloak/keycloak#16081 for why this is happening) Actual fix is to disable the refresh token rotation in KC.. but then you get exposed to other security risks. |
@Badisi I think this must be a different problem since i send the refresh token and get back a new set of valid tokens. However, the auth_time of the tokens i receive does not match the original auth_time anymore, which leads to above error raised within oidc-client-ts |
@claria, the issue mentioned by @pamapa is also about refresh token rotation. I've made it successfully working, using this demo app.
|
Thanks for your support. I set Keycloak up in your demo app.
|
Just to be sure i tried it as well with the "Revoke Refresh Token" setting enabled.
|
Hi, As i mentioned in my inital post, we have Azure B2C as Identity Provider in Keycloak. When i do NOT use Azure B2C as Identity provider but a local user login, everything works. As soon as a login to keycloak via the azure b2c, the silent login does not work anymore. |
Arghh. The issue was a simple configuration error. Maybe this helps somebody sometimes. Thank you very much for your help and time!!! |
Hi,
When a user has a single browser tab with the application open, all works fine. Silent Refresh works etc.
The problems begin the user opens a second tab:
As you can see, the silentrefresh from tab 1 yields an id token with the auth_time from tab 2 and thus fails. What are we doing wrong?
I think, this is a basic configuration issue on our side and not a bug.
Thanks for your help! (And this awesome library)
The text was updated successfully, but these errors were encountered: