Merge pull request #1858 from erikmiller-gusto/main

stefanprodan · web-flow · commit 9090802dcc59 · 2025-12-18T09:09:51.000+02:00
fix: datadog metrics should provide http status code on error if non-2xx response
diff --git a/pkg/metrics/providers/datadog.go b/pkg/metrics/providers/datadog.go
@@ -131,7 +131,7 @@ func (p *DatadogProvider) RunQuery(query string) (float64, error) {
 	}
 
 	if r.StatusCode != http.StatusOK {
-		return 0, fmt.Errorf("error response: %s: %w", string(b), err)
+		return 0, fmt.Errorf("error response: %s: %s", string(b), r.Status)
 	}
 
 	var res datadogResponse

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ func (p *DatadogProvider) RunQuery(query string) (float64, error) {`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`if r.StatusCode != http.StatusOK {`
`134`		`- return 0, fmt.Errorf("error response: %s: %w", string(b), err)`
	`134`	`+ return 0, fmt.Errorf("error response: %s: %s", string(b), r.Status)`
`135`	`135`	`}`
`136`	`136`
`137`	`137`	`var res datadogResponse`