-
Notifications
You must be signed in to change notification settings - Fork 927
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
Built-in auto token refresh #148
Comments
@robsontenorio Really looking forward to this. Any idea when you'll have time for the PR? |
Just waiting another PR be merged. |
Done! PR sent. |
@robsontenorio any progress on the issue? I am trying to solve the same problem. |
It seem was not well received. Just waiting a response. |
Any news on this? |
There was some tries in the past, but mantainers does not reviewed it. The code base from PRs is deprecated now. |
What is the current approach of most people to solve this? I also need this functionality. |
we're using this workaround for now. #208 (comment). would like to know if this project is moving forward anymore? there is some pull requests merged but for some reason there's no releases since a while. |
Any update on this this issue ? |
Any news? |
Nothing? |
Any update for the refresh token support? |
Anything? |
Hey there! We are working on refresh support! For now you can use the new branch (feat/refresh) Docs and more details can be found at #361 |
Is this functionality available in the master branch? |
No, sorry, this is only available in the auth-next branch. If you need this feature, you should upgrade. See the status page for more details. Closing this issue since it's available now :) |
What problem does this feature solve?
Automatically refresh the "token" if it expires, because some Oauth providers has a short token TTL.
What does the proposed changes look like?
Once #145 has been accepted, I will propose a PR to auto refresh the "token" if it has been expired.
@pi0 @breakingrobot where place it? Maybe
lib/core/auth.js
?The text was updated successfully, but these errors were encountered: