SendWithAuth requires GetAccessToken call explicitly

[nam-truong-le]

func (c *Client) SendWithAuth(req *http.Request, v interface{}) error {
	// c.Lock()
	c.mu.Lock()
	// Note: Here we do not want to `defer c.Unlock()` because we need `c.Send(...)`
	// to happen outside of the locked section.

	if c.Token != nil {
		if !c.tokenExpiresAt.IsZero() && c.tokenExpiresAt.Sub(time.Now()) < RequestNewTokenBeforeExpiresIn {
			// c.Token will be updated in GetAccessToken call
			if _, err := c.GetAccessToken(req.Context()); err != nil {
				// c.Unlock()
				c.mu.Unlock()
				return err
			}
		}

		req.Header.Set("Authorization", "Bearer "+c.Token.Token)
	}
	// Unlock the client mutex before sending the request, this allows multiple requests
	// to be in progress at the same time.
	// c.Unlock()
	c.mu.Unlock()
	return c.Send(req, v)
}

Can we make this easier to use. This method doesn't fetch token automatically in case there is no token. Thank you.

[plutov]

@nam-truong-le feel free to provide a PR.

[buffos]

All methods require fetching the token as far as I can tell. You can either do that explicitly every time, or include it in your NewClient function.