-
Notifications
You must be signed in to change notification settings - Fork 92
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
[Bug] Expired token unable to refresh #68
Comments
Ill keep an eye on this. I think #70 might have fixed this? This error often happens after the connection leaked issue happens. |
#70 did NOT fix this. Still seeing it. doesnt happen on every request. so dont really know what it could be. |
Here are my thoughts
however in this code we see github-java-client/src/main/java/com/spotify/github/v3/clients/JwtTokenIssuer.java Line 77 in 12f5323
so i am wondering if the fix is simply to remove 60 seconds. |
fixes spotify#68 per example in related issue; time sync issue can occur when setting issued at to be the current time. This sets the issued at to be 1 minute earlier to avoid this. Tested locally for 2 days with no issues.
on restart of the application; everything works fine.
The text was updated successfully, but these errors were encountered: