starred/godoxy-yusing
1
0
Fork 0
mirror of https://github.com/yusing/godoxy.git synced 2026-03-30 13:51:52 +02:00
Code Issues 18 Packages Projects Releases 10 Wiki Activity

refactor(http): enhance health check error logic by treating all 5xx as unhealthy

Browse Source
This commit is contained in:
yusing
2025-12-17 12:24:04 +08:00
parent b49d565213
commit 4299c78067
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
internal/watcher/health/monitor/http.go
View File

@@ -62,8 +62,7 @@ func (mon *HTTPHealthMonitor) CheckHealth() (types.HealthCheckResult, error) {
respErr := pinger.DoTimeout(req, resp, mon.config.Timeout)
lat := time.Since(start)
switch {
case respErr != nil:
if respErr != nil {
// treat TLS error as healthy
var tlsErr *tls.CertificateVerificationError
if ok := errors.As(respErr, &tlsErr); !ok {
@@ -72,7 +71,9 @@ func (mon *HTTPHealthMonitor) CheckHealth() (types.HealthCheckResult, error) {
Detail: respErr.Error(),
}, nil
}
case resp.StatusCode() == fasthttp.StatusServiceUnavailable:
}
if status := resp.StatusCode(); status >= 500 && status < 600 {
return types.HealthCheckResult{
Latency: lat,
Detail: fasthttp.StatusMessage(resp.StatusCode()),