Skip to content
This repository has been archived by the owner on Nov 20, 2020. It is now read-only.

Don't auth if there is an existing bearer token #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

markpeek
Copy link
Contributor

The sdk client was requesting a new bearer token for each api call. This change will look for and use an existing bearer token prior to going through the authorization.

@vmwclabot
Copy link

@markpeek, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

Copy link
Contributor

@Prativa20 Prativa20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should check the token expiry timestamp as well, so that we don't use the existing token that is expired.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants