Merge pull request #1461 from cloudflare/ensure-get-ips-is-200

ips: handle non-200 responses
cloudflare · Dec 13, 2023 · cf73155 · cf73155
2 parents fd17f69 + 44feda7
commit cf73155
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/ips.go b/ips.go
@@ -1,6 +1,7 @@
 package cloudflare
 
 import (
+	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -41,6 +42,9 @@ func IPs() (IPRanges, error) {
 	if err != nil {
 		return IPRanges{}, fmt.Errorf("HTTP request failed: %w", err)
 	}
+	if resp.StatusCode != http.StatusOK {
+		return IPRanges{}, errors.New("HTTP request failed: status is not HTTP 200")
+	}
 	defer resp.Body.Close()
 	body, err := io.ReadAll(resp.Body)
 	if err != nil {