Skip to content

Commit 48c8be9

Browse files
authored
fix(posture/zones): correctly handle api errors (#637)
1 parent bf4e4b0 commit 48c8be9

File tree

4 files changed

+16
-0
lines changed

4 files changed

+16
-0
lines changed

sysdig/internal/client/v2/posture_controls.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,9 @@ func (c *Client) GetPostureControl(ctx context.Context, id int64) (*PostureContr
4848
}
4949
defer response.Body.Close()
5050

51+
if response.StatusCode != http.StatusOK {
52+
return nil, c.ErrorFromResponse(response)
53+
}
5154
wrapper, err := Unmarshal[SaveControlResponse](response.Body)
5255
if err != nil {
5356
return nil, err

sysdig/internal/client/v2/posture_policies.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,10 @@ func (client *Client) ListPosturePolicies(ctx context.Context) ([]PosturePolicy,
2828
}
2929
defer response.Body.Close()
3030

31+
if response.StatusCode != http.StatusOK {
32+
return nil, client.ErrorFromResponse(response)
33+
}
34+
3135
resp, err := Unmarshal[PostureZonePolicyListResponse](response.Body)
3236
if err != nil {
3337
return nil, err

sysdig/internal/client/v2/posture_zones.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,9 @@ func (client *Client) GetPostureZone(ctx context.Context, id int) (*PostureZone,
5454
}
5555
defer response.Body.Close()
5656

57+
if response.StatusCode != http.StatusOK {
58+
return nil, client.ErrorFromResponse(response)
59+
}
5760
wrapper, err := Unmarshal[PostureZoneResponse](response.Body)
5861
if err != nil {
5962
return nil, err

sysdig/internal/client/v2/zones.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ func (client *Client) GetZones(ctx context.Context, name string) ([]Zone, error)
3131
}
3232
defer response.Body.Close()
3333

34+
if response.StatusCode != http.StatusOK {
35+
return nil, client.ErrorFromResponse(response)
36+
}
3437
wrapper, err := Unmarshal[ZonesWrapper](response.Body)
3538
if err != nil {
3639
return nil, err
@@ -46,6 +49,9 @@ func (client *Client) GetZoneById(ctx context.Context, id int) (*Zone, error) {
4649
}
4750
defer response.Body.Close()
4851

52+
if response.StatusCode != http.StatusOK {
53+
return nil, client.ErrorFromResponse(response)
54+
}
4955
zone, err := Unmarshal[Zone](response.Body)
5056
if err != nil {
5157
return nil, err

0 commit comments

Comments
 (0)