diff --git a/app/vmalert/web.go b/app/vmalert/web.go index 77a06aa0d..ec24a3edb 100644 --- a/app/vmalert/web.go +++ b/app/vmalert/web.go @@ -211,7 +211,7 @@ func (rh *requestHandler) groups() []APIGroup { rh.m.groupsMu.RLock() defer rh.m.groupsMu.RUnlock() - var groups []APIGroup + groups := make([]APIGroup, 0) for _, g := range rh.m.groups { groups = append(groups, g.toAPI()) } @@ -276,6 +276,7 @@ func (rh *requestHandler) listAlerts() ([]byte, error) { defer rh.m.groupsMu.RUnlock() lr := listAlertsResponse{Status: "success"} + lr.Data.Alerts = make([]*APIAlert, 0) for _, g := range rh.m.groups { for _, r := range g.Rules { a, ok := r.(*AlertingRule) diff --git a/app/vmalert/web_test.go b/app/vmalert/web_test.go index 600010527..3a70f6fef 100644 --- a/app/vmalert/web_test.go +++ b/app/vmalert/web_test.go @@ -162,5 +162,59 @@ func TestHandler(t *testing.T) { t.Run("/api/v1/1/0/status", func(t *testing.T) { getResp(ts.URL+"/api/v1/1/0/status", nil, 404) }) - +} + +func TestEmptyResponse(t *testing.T) { + rh := &requestHandler{m: &manager{groups: make(map[uint64]*Group)}} + ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { rh.handler(w, r) })) + defer ts.Close() + + getResp := func(url string, to interface{}, code int) { + t.Helper() + resp, err := http.Get(url) + if err != nil { + t.Fatalf("unexpected err %s", err) + } + if code != resp.StatusCode { + t.Errorf("unexpected status code %d want %d", resp.StatusCode, code) + } + defer func() { + if err := resp.Body.Close(); err != nil { + t.Errorf("err closing body %s", err) + } + }() + if to != nil { + if err = json.NewDecoder(resp.Body).Decode(to); err != nil { + t.Errorf("unexpected err %s", err) + } + } + } + + t.Run("/api/v1/alerts", func(t *testing.T) { + lr := listAlertsResponse{} + getResp(ts.URL+"/api/v1/alerts", &lr, 200) + if lr.Data.Alerts == nil { + t.Errorf("expected /api/v1/alerts response to have non-nil data") + } + + lr = listAlertsResponse{} + getResp(ts.URL+"/vmalert/api/v1/alerts", &lr, 200) + if lr.Data.Alerts == nil { + t.Errorf("expected /api/v1/alerts response to have non-nil data") + } + }) + + t.Run("/api/v1/rules", func(t *testing.T) { + lr := listGroupsResponse{} + getResp(ts.URL+"/api/v1/rules", &lr, 200) + if lr.Data.Groups == nil { + t.Errorf("expected /api/v1/rules response to have non-nil data") + } + + lr = listGroupsResponse{} + getResp(ts.URL+"/vmalert/api/v1/rules", &lr, 200) + if lr.Data.Groups == nil { + t.Errorf("expected /api/v1/rules response to have non-nil data") + } + }) } diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md index 8e1a3d58e..f08c2f155 100644 --- a/docs/CHANGELOG.md +++ b/docs/CHANGELOG.md @@ -40,6 +40,7 @@ The following tip changes can be tested by building VictoriaMetrics components f * BUGFIX: [vmui](https://docs.victoriametrics.com/#vmui): fix the display of the tenant selector. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4160). * BUGFIX: [vmui](https://docs.victoriametrics.com/#vmui): fix issue where vmui would freeze when adding a query that returned regular rows alongside a heatmap query. * BUGFIX: [vmalert](https://docs.victoriametrics.com/vmalert.html): properly display an error when using `query` function for templating value of `-external.alert.source` flag. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4181). +* BUGFIX: [vmalert](https://docs.victoriametrics.com/vmalert.html): properly return empty slices instead of nil for `/api/v1/rules` and `/api/v1/alerts` API handlers. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4221). ## [v1.90.0](https://github.com/VictoriaMetrics/VictoriaMetrics/releases/tag/v1.90.0)