-
Notifications
You must be signed in to change notification settings - Fork 107
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
Proposal: create Account from refreshing oauth2.TokenSource #29
Comments
@sethterashima can I ask you to look into this issue? Its still unclear how to create long-running applications to handle access token expiry. It seems the idiomatic way doing this would be using an |
Friendly ping. This is an absolut blocker for any long-running application. How are applications supposed to perform token refresh with the current capabilities? |
@andig |
@fabianhu that's exactly what a Golang Imho this proposal is a more go-ish and fully-featured way of doing things. |
Currently I can only refresh tokens during +/- 24h ... after that, refreshing token returns 401 and I'm forced to login again to get a new CODE and then a new token/refresh_token Does anyone know if I'm doing something wrong? @andig are you aware of this issue? |
When refreshing access tokens make sure to store the new refresh token, too. |
Creating account from token string and saving tokens inside the account is brittle since tokens expire. Instead, create
Account
by passing either anoauth2.Token
oroauth2.TokenSource
.Using #24, these tokens would then be refreshed.
The text was updated successfully, but these errors were encountered: