Skip to content

Add support for 500 and 429 status codes #620

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
agners merged 2 commits into from
Feb 2, 2026
+6 −2
Merged

Add support for 500 and 429 status codes #620

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4f3ad6f
Add support for 500 and 429 status codes
mdegat01 Dec 10, 2025
a0faa56
Add missing quote
sairon Feb 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions client/client.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ import (
"net/http"
"time"

log "github.com/sirupsen/logrus"
resty "github.com/go-resty/resty/v2"
log "github.com/sirupsen/logrus"
)

// RawJSON controls if the client does json handling or outputs it raw
Expand All @@ -21,11 +21,15 @@ func GenericJSONErrorHandling(resp *resty.Response, err error) (*resty.Response,
// Handle known error codes as well as error codes which Supervisor returns without
// a (JSON) body.
switch resp.StatusCode() {
case http.StatusOK, http.StatusBadRequest, http.StatusNotFound, http.StatusServiceUnavailable:
case http.StatusOK, http.StatusBadRequest, http.StatusNotFound, http.StatusServiceUnavailable, http.StatusTooManyRequests:
// Success and these errors should have JSON responses
if !resty.IsJSONType(resp.Header().Get("Content-Type")) {
return nil, fmt.Errorf("unexpected non-JSON response (status: %d)", resp.StatusCode())
}
case http.StatusInternalServerError:
if !resty.IsJSONType(resp.Header().Get("Content-Type")) {
return nil, fmt.Errorf("unknown error occurred, check supervisor logs with 'ha supervisor logs")
}
Comment thread
sairon marked this conversation as resolved.
case http.StatusUnauthorized:
if !resty.IsJSONType(resp.Header().Get("Content-Type")) {
return nil, fmt.Errorf("unauthorized: missing or invalid API token")
Expand Down