-
Notifications
You must be signed in to change notification settings - Fork 836
Support for Code Flow PKCE with Refresh tokens #1362
Comments
It supports refresh tokens and renewal already. |
Cool, how can we configure to use |
Request offline_access as a scope, as per the spec. https://openid.net/specs/openid-connect-core-1_0.html#OfflineAccess |
@brockallen Perfect, thanks a lot. It works when using Could you please mention it in the Wiki documentation? |
You know refresh tokens in the browser are not really recommended, though, right? |
That's what I thought at first. See https://auth0.com/docs/tokens/refresh-tokens#for-single-page-apps |
Did you watch the video above? |
I just watched the video. Thanks for sharing. Could you please explain how silent renewal with iFrame is still better than refresh token rotation? |
@Keth-dev Were you able to get enlightened on that? I was wondering about the same. From what I understand from the video there is no way to avoid browser vulnerabilities aside from using BFF, which essentially means once someone busts your SPA with XSRF it doesn't really matter whether you use refresh token rotation or silent renewal. |
The difference is the amount of damage in what amount of time. A leaked refresh token will allow an attacker much longer and potentially infinite time if there's no absolute limit on the RT. |
The difference is the amount of damage in what amount of time. A leaked refresh token will allow an attacker much longer and potentially infinite time if there's no absolute limit on the RT. The BFF approach we suggest (if you're using ASP.NET) is this one: https://blog.duendesoftware.com/posts/20210326_bff/ |
@brockallen I see, but once the length of sessions kept with the ID provider for silent renewal surpasses refresh token lifetime it should be virtually the same if I am not mistaken. |
As far as I see, the
oidc-client-js
library supports only silent renewal with a silent renew callback URL.Is there a way to make it work with refresh token mechanism? If not, are there any plans to support it in the future?
The text was updated successfully, but these errors were encountered: