fix(extension): consider requests failed when status code is not 2xx (#1658)

pkarakal · web-flow · commit e6b69d71e430 · 2025-01-14T08:25:40.000-08:00
diff --git a/collector/internal/extensionapi/client.go b/collector/internal/extensionapi/client.go
@@ -180,7 +180,7 @@ func (e *Client) doRequest(req *http.Request, out interface{}) (*http.Response,
 	if err != nil {
 		return nil, err
 	}
-	if resp.StatusCode != 200 {
+	if resp.StatusCode < 200 || resp.StatusCode > 299 {
 		return nil, fmt.Errorf("request failed with status %s", resp.Status)
 	}
 	defer resp.Body.Close()

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ func (e Client) doRequest(req http.Request, out interface{}) (*http.Response,`
`180`	`180`	`if err != nil {`
`181`	`181`	`return nil, err`
`182`	`182`	`}`
`183`		`- if resp.StatusCode != 200 {`
	`183`	`+ if resp.StatusCode < 200 \|\| resp.StatusCode > 299 {`
`184`	`184`	`return nil, fmt.Errorf("request failed with status %s", resp.Status)`
`185`	`185`	`}`
`186`	`186`	`defer resp.Body.Close()`