Merge pull request #22087 from vespa-engine/mpolden/check-status

Check status code from GitHub API

1 files changed, 5 insertions, 1 deletions

diff --git a/client/go/cond_make.go b/client/go/cond_make.go
index bddc80aeeef..4fba3efcb16 100644
--- a/client/go/cond_make.go
+++ b/client/go/cond_make.go
@@ -84,11 +84,15 @@ func latestTag() (string, error) {
 func latestReleasedTag(mirror string) (string, error) {
 	switch mirror {
 	case "github":
-		resp, err := http.Get("https://api.github.com/repos/vespa-engine/vespa/releases/latest")
+		url := "https://api.github.com/repos/vespa-engine/vespa/releases/latest"
+		resp, err := http.Get(url)
 		if err != nil {
 			return "", err
 		}
 		defer resp.Body.Close()
+		if resp.StatusCode != http.StatusOK {
+			return "", fmt.Errorf("got status %d from %s", resp.StatusCode, url)
+		}
 		var release gitHubRelease
 		dec := json.NewDecoder(resp.Body)
 		if err := dec.Decode(&release); err != nil {