fix(artifacts): Make HttpCredentials fetchUrl throws exception on non…

…-successful response (#3946) (#3953)
spinnaker · Aug 16, 2019 · 411afdd · 411afdd
1 parent d0d32ce
commit 411afdd
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 0 deletions.
diff --git a/.../java/com/netflix/spinnaker/clouddriver/artifacts/config/BaseHttpArtifactCredentials.java b/.../java/com/netflix/spinnaker/clouddriver/artifacts/config/BaseHttpArtifactCredentials.java
@@ -76,6 +76,10 @@ protected ResponseBody fetchUrl(HttpUrl url) throws IOException {
     Request request = new Request.Builder().headers(headers).url(url).build();
 
     Response downloadResponse = okHttpClient.newCall(request).execute();
+    if (!downloadResponse.isSuccessful()) {
+      throw new IOException(
+          String.format("Received %d status code from %s", downloadResponse.code(), url.host()));
+    }
     return downloadResponse.body();
   }
 }
diff --git a/...st/java/com/netflix/spinnaker/clouddriver/artifacts/http/HttpArtifactCredentialsTest.java b/...st/java/com/netflix/spinnaker/clouddriver/artifacts/http/HttpArtifactCredentialsTest.java
@@ -18,6 +18,7 @@
 
 import static com.github.tomakehurst.wiremock.client.WireMock.*;
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.catchThrowable;
 
 import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.client.MappingBuilder;
@@ -73,6 +74,24 @@ void downloadWithNoAuth(@WiremockResolver.Wiremock WireMockServer server) throws
     runTestCase(server, account, m -> m.withHeader("Authorization", absent()));
   }
 
+  @Test
+  void throwExceptionOnNonSuccessfulResponse(@WiremockResolver.Wiremock WireMockServer server) {
+    HttpArtifactAccount account = new HttpArtifactAccount();
+    HttpArtifactCredentials credentials = new HttpArtifactCredentials(account, okHttpClient);
+    Artifact artifact =
+        Artifact.builder().reference(server.baseUrl() + URL).type("http/file").build();
+    account.setName("my-http-account");
+    server.stubFor(any(urlPathEqualTo(URL)).willReturn(aResponse().withStatus(404)));
+
+    Throwable thrown = catchThrowable(() -> credentials.download(artifact));
+
+    assertThat(thrown)
+        .isInstanceOf(IOException.class)
+        .hasMessageContaining("404")
+        .hasMessageContaining(server.baseUrl());
+    assertThat(server.findUnmatchedRequests().getRequests()).isEmpty();
+  }
+
   private void runTestCase(
       WireMockServer server,
       HttpArtifactAccount account,