Consider any HTTP response code > 399 as an error (#3834)

elastic · May 7, 2024 · 88ec185 · 88ec185
1 parent 19b3c81
commit 88ec185
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/pkg/downloads/releases.go b/pkg/downloads/releases.go
@@ -103,7 +103,7 @@ func (r *ArtifactURLResolver) Resolve() (string, string, error) {
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode > 399 {
 			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
 		}
 
@@ -242,7 +242,7 @@ func (as *ArtifactsSnapshotVersion) GetSnapshotArtifactVersion(project string, v
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode > 399 {
 			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
 		}
 
@@ -380,7 +380,7 @@ func (asur *ArtifactsSnapshotURLResolver) Resolve() (string, string, error) {
 			return backoff.Permanent(err)
 		}
 
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode > 399 {
 			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
 		}
 
@@ -497,7 +497,7 @@ func (r *ReleaseURLResolver) Resolve() (string, string, error) {
 		defer resp.Body.Close()
 		_, _ = io.Copy(io.Discard, resp.Body)
 
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode > 399 {
 			return backoff.Permanent(fmt.Errorf("not found for url %s", url))
 		}
 

diff --git a/pkg/downloads/versions.go b/pkg/downloads/versions.go
@@ -193,7 +193,7 @@ func GetElasticArtifactVersion(version string) (string, error) {
 			return fmt.Errorf("error getting %s: %w", url, err)
 		}
 
-		if resp.StatusCode == http.StatusNotFound {
+		if resp.StatusCode > 399 {
 			return backoff.Permanent(fmt.Errorf("version %s not found at %s", version, url))
 		}