Add a check for http response code

aws · Oct 4, 2022 · 2e05d8f · 2e05d8f
1 parent 0ae333e
commit 2e05d8f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/agent/api/appnet/client_linux.go b/agent/api/appnet/client_linux.go
@@ -26,6 +26,7 @@ import (
 	"github.com/aws/amazon-ecs-agent/agent/logger"
 	"github.com/aws/amazon-ecs-agent/agent/logger/field"
 	"github.com/aws/amazon-ecs-agent/agent/utils/retry"
+	"github.com/pkg/errors"
 	prometheus "github.com/prometheus/client_model/go"
 )
 
@@ -42,6 +43,9 @@ func (cl *client) GetStats(config serviceconnect.RuntimeConfig) (map[string]*pro
 		return nil, err
 	}
 	defer resp.Body.Close()
+	if resp.StatusCode != http.StatusOK {
+		return nil, errors.Wrapf(err, "received non-OK HTTP status %v from Service Connect stats endpoint", resp.StatusCode)
+	}
 	return parseServiceConnectStats(resp.Body)
 }