idpy-oidc icon indicating copy to clipboard operation
idpy-oidc copied to clipboard

Does this rp library support refresh_token?

Open johnbyrne7 opened this issue 2 years ago • 2 comments

I dont think this (rp) library supports refresh token. In rp_handler.py, method get_access_and_id_token, this line does get a refresh token from the op, in token_resp , when 'offline_access' is added to scopes.

779: token_resp = self.get_tokens(state, client=client)

However, the refresh token is not returned in the dictionary returned by that method, so I dont see how refresh token could be supported

787: return {"access_token": access_token, "id_token": id_token}

am I missing something? any direction would be appreciated

johnbyrne7 avatar Jun 02 '23 21:06 johnbyrne7

Lower layers of the library do support refresh tokens. If rp_handler.py doesn't I have change that.

rohe avatar Jul 18 '23 08:07 rohe

Added refresh_token to what get_access_and_id_token() returns. Presently in the example_fix branch.

rohe avatar Jul 18 '23 09:07 rohe