diff --git a/internal/api/user.go b/internal/api/user.go index a81eb029..399c50ae 100644 --- a/internal/api/user.go +++ b/internal/api/user.go @@ -80,7 +80,6 @@ func (api *RestAPI) getUsers(rw http.ResponseWriter, r *http.Request) { // @security ApiKeyAuth // @router /api/user/{id} [post] func (api *RestAPI) updateUserByRequest(rw http.ResponseWriter, r *http.Request) { - if user := repository.GetUserFromContext(r.Context()); !user.HasRole(schema.RoleAdmin) { handleError(fmt.Errorf("only admins are allowed to update a user"), http.StatusForbidden, rw) return diff --git a/pkg/metricstore/level.go b/pkg/metricstore/level.go index 80df92aa..1737a774 100644 --- a/pkg/metricstore/level.go +++ b/pkg/metricstore/level.go @@ -382,8 +382,8 @@ func (l *Level) findLevel(selector []string) *Level { // - selector: Pattern to match (consumed recursively) // - offset: Metric index in metrics slice (from MetricConfig.offset) // - f: Callback invoked on each matching buffer; path holds the matched -// level keys (e.g. ["cluster","node001","cpu0"]) so callers can -// identify which node/component the buffer belongs to +// level keys (e.g. ["cluster","node001","cpu0"]) so callers can +// identify which node/component the buffer belongs to // - path: Accumulated level keys from the root; pass nil at the top level // // Returns: diff --git a/pkg/metricstore/prune_test.go b/pkg/metricstore/prune_test.go index fca5329e..f316b138 100644 --- a/pkg/metricstore/prune_test.go +++ b/pkg/metricstore/prune_test.go @@ -104,8 +104,8 @@ func TestFreeViaProviderPrunesDeadNodes(t *testing.T) { const freq int64 = 60 thr := time.Unix(100000, 0) - writeNode(ms, "fritz", "dead", nil, freq, 1000, 2000) // stale, not used -> pruned - writeNode(ms, "fritz", "busy", nil, freq, 1000, 2000) // stale, but used -> kept + writeNode(ms, "fritz", "dead", nil, freq, 1000, 2000) // stale, not used -> pruned + writeNode(ms, "fritz", "busy", nil, freq, 1000, 2000) // stale, but used -> kept writeNode(ms, "fritz", "live", nil, freq, 100000, 100600) // fresh -> kept ms.SetNodeProvider(&fakeNodeProvider{nodes: map[string][]string{"fritz": {"busy"}}})