fix(appDelete): Handle retrofit exceptions with no response object. (s…

…pinnaker#2605)
ziggyw · Jan 17, 2019 · 1bdd044 · 1bdd044
1 parent 6f2b81e
commit 1bdd044
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/...m/netflix/spinnaker/orca/applications/tasks/VerifyApplicationHasNoDependenciesTask.groovy b/...m/netflix/spinnaker/orca/applications/tasks/VerifyApplicationHasNoDependenciesTask.groovy
@@ -59,7 +59,10 @@ class VerifyApplicationHasNoDependenciesTask implements Task {
         existingDependencyTypes << "security groups"
       }
     } catch (RetrofitError e) {
-      if (e.response?.status != 404) {
+      if (!e.response) {
+        def exception = [operation: stage.tasks[-1].name, reason: e.message]
+        return new TaskResult(ExecutionStatus.TERMINAL, [exception: exception])
+      } else if (e.response && e.response.status && e.response.status != 404) {
         def resp = e.response
         def exception = [statusCode: resp.status, operation: stage.tasks[-1].name, url: resp.url, reason: resp.reason]
         try {