From 90ea93c1bedb98ecf7067360f020933300646e00 Mon Sep 17 00:00:00 2001
From: Denis Rechkunov <denis.rechkunov@elastic.co>
Date: Wed, 8 May 2024 11:24:51 +0200
Subject: [PATCH] Report unexpected status codes as errors

The only expected code is 200, the rest should fail and the error
message should contain the status code itself.
---
 pkg/downloads/releases.go | 16 ++++++++--------
 pkg/downloads/versions.go |  4 ++--
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/pkg/downloads/releases.go b/pkg/downloads/releases.go
index 92a963765..a10acb05f 100644
--- a/pkg/downloads/releases.go
+++ b/pkg/downloads/releases.go
@@ -103,8 +103,8 @@ func (r *ArtifactURLResolver) Resolve() (string, string, error) {
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode > 399 {
-			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
+		if resp.StatusCode != http.StatusOK {
+			return backoff.Permanent(fmt.Errorf("unexpected status code %d from url %s", resp.StatusCode, url))
 		}
 
 		return nil
@@ -242,8 +242,8 @@ func (as *ArtifactsSnapshotVersion) GetSnapshotArtifactVersion(project string, v
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode > 399 {
-			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
+		if resp.StatusCode != http.StatusOK {
+			return backoff.Permanent(fmt.Errorf("unexpected status code %d from url %s", resp.StatusCode, url))
 		}
 
 		return nil
@@ -380,8 +380,8 @@ func (asur *ArtifactsSnapshotURLResolver) Resolve() (string, string, error) {
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode > 399 {
-			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
+		if resp.StatusCode != http.StatusOK {
+			return backoff.Permanent(fmt.Errorf("unexpected status code %d from url %s", resp.StatusCode, url))
 		}
 
 		return nil
@@ -497,8 +497,8 @@ func (r *ReleaseURLResolver) Resolve() (string, string, error) {
 		defer resp.Body.Close()
 		_, _ = io.Copy(io.Discard, resp.Body)
 
-		if resp.StatusCode > 399 {
-			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
+		if resp.StatusCode != http.StatusOK {
+			return backoff.Permanent(fmt.Errorf("unexpected status code %d from url %s", resp.StatusCode, url))
 		}
 
 		found = true
diff --git a/pkg/downloads/versions.go b/pkg/downloads/versions.go
index 2fec0009c..427f5db08 100644
--- a/pkg/downloads/versions.go
+++ b/pkg/downloads/versions.go
@@ -193,8 +193,8 @@ func GetElasticArtifactVersion(version string) (string, error) {
 			return fmt.Errorf("error getting %s: %w", url, err)
 		}
 
-		if resp.StatusCode > 399 {
-			return backoff.Permanent(fmt.Errorf("version %s not found at %s", version, url))
+		if resp.StatusCode != http.StatusOK {
+			return backoff.Permanent(fmt.Errorf("unexpected status code %d from url %s when fetching version %s", resp.StatusCode, url, version))
 		}
 
 		defer resp.Body.Close()