Skip to content
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

feat: Add ability to refresh token automatically before expiration. #227

Closed
wants to merge 20 commits into from

Conversation

nolanmar511
Copy link
Contributor

This pull request makes it possible to refresh the token when it is not expired but will expire in some period of time.
This can be used to address the issues described in #133, by allowing users to specify that tokens should be refreshed if they will expire soon. This is also necessary to fix googleapis/cloud-profiler-nodejs#94

For earlier discussion of these changes, see PR #208.

ofrobots and others added 20 commits November 20, 2017 15:01
…leapis#202)

BREAKING CHANGE: The IAMAuth, Compute, JWT, JWTAccess, OAuth2, and UserRefreshClient types are no longer available as public properties on the GoogleAuth object.  They are now exported directly from the top level module.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 3, 2018
@JustinBeckwith
Copy link
Contributor

Thanks! We need to land #230 first.

@JustinBeckwith
Copy link
Contributor

@nolanmar511 So sorry for all of the churn here. We're finally done with all of the crazy, and you should be free to get this moving again. Now we can rebase this against the master branch, which is mostly what we had in next. I ran into some trouble porting my PRs, and ended up using git format-patch and git apply to port mine over.

@nolanmar511
Copy link
Contributor Author

Now rebasing against master. See #242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants