stacklok / toolhive / 33897715598

70%

Build Type

push

github

Committed by web-flow

Commit Message

Restart health checks when a backend's auth config changes (#6510)

backendChanged compared only BaseURL and TransportType, so UpdateBackends never restarted a check goroutine when a backend gained or changed outgoing auth.
The stale auth-less copy kept probing tokenless, the 401 classified BackendUnauthenticated instead of the expected-auth-challenge healthy, and ShouldAdvertise dropped the backend from aggregation until a pod restart.
Comparing AuthConfig in backendChanged restarts the loop so probes classify against the current auth config.
Includes the red-green regression test plus backendChanged table cases.

Coverage Stats

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

3 existing lines in 2 files now uncovered.

77757 of 111125 relevant lines covered (69.97%)

92.46 hits per line