stacklok / toolhive / 21590795175

61%

Build Type

push

github

Committed by web-flow

Commit Message

Fix OAuth token refresh context cancellation (#3539)

* Fix OAuth token refresh context cancellation

The TokenSource created in processToken() was using the OAuth flow's
startup context, which gets cancelled when the callback server shuts down.
When the access token expires (~1 hour later) and refresh is attempted,
the HTTP request fails with "context canceled".

This triggers a full re-authentication flow with new DCR client
registration, requiring user interaction in the browser.

The fix uses context.Background() for the TokenSource, since it is
long-lived and must remain valid for token refresh operations long
after the OAuth flow completes.

Closes gh-3538

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Fix unused ctx parameter lint error

---------

Signed-off-by: Greg Katz <gkatz@indeed.com>
Co-authored-by: Muhammad Amir Ejaz <amir@stacklok.com>

Run Details

8 of 8 new or added lines in 1 file covered. (100.0%)

5 existing lines in 2 files now uncovered.

39750 of 65675 relevant lines covered (60.53%)

76.37 hits per line