search

IdentityPython / idpy-oidc

Implementation of everything OIDC and OAuth2
Apache License 2.0
40 stars 22 forks source link

Does this rp library support refresh_token? #68

Open johnbyrne7 opened 1 year ago

johnbyrne7 commented 1 year ago

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

rohe commented 1 year ago

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

rohe commented 1 year ago

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