Skip to content

Commit

Permalink
Token issues (#1955)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
edobbsskylark committed Nov 7, 2022
1 parent df07026 commit 95767d4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/usage.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ public class TwitterAuthenticator : AuthenticatorBase {
}

protected override async ValueTask<Parameter> GetAuthenticationParameter(string accessToken) {
var token = string.IsNullOrEmpty(Token) ? await GetToken() : Token;
return new HeaderParameter(KnownHeaders.Authorization, token);
Token = string.IsNullOrEmpty(Token) ? await GetToken() : Token;
return new HeaderParameter(KnownHeaders.Authorization, Token);
}
}
```
Expand Down

0 comments on commit 95767d4

Please sign in to comment.