Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While following this page and using it a lot while learning APIs, I had an issue where every request I was making to my client, it would reach out to get a new token. I assume this is not intended, and this was my fix for it. The local variable 'token' in the GetAuthenticationParameter function doesn't live outside of this and there for the obtained bearer token is never saved to the client object. I just forced the gettoken function to save to the Token variable that is in AuthenticatorBase.
Please let me know if I am missing something as I would love to learn!
I do figure that this does not take into account "what if the bearer token expires?" And to be honest I have no clue how this is handled! I plan on doing some checks if the request comes back with unauthorized, to get a new token, but if you have ideas or know of better ways to handle bearer tokens expiring please let me know! I am new to working with APIs.
Description
Purpose
This pull request is a:
Checklist