Merge pull request #297 from james-bright-helix/do-not-use-http-clien…

…t-retry-for-wait-url

Disable Apache HTTP Client retry in WaitUtil

src/main/java/org/jolokia/docker/maven/util/WaitUtil.java

@@ -10,6 +10,7 @@
 import org.apache.http.client.methods.CloseableHttpResponse;
 import org.apache.http.client.methods.RequestBuilder;
 import org.apache.http.impl.client.CloseableHttpClient;
+import org.apache.http.impl.client.DefaultHttpRequestRetryHandler;
 import org.apache.http.impl.client.HttpClientBuilder;
 
 /**
@@ -34,6 +35,9 @@ public class WaitUtil {
  public static final int DEFAULT_MIN_STATUS = 200;
  public static final int DEFAULT_MAX_STATUS = 399;
 
+ // Disable HTTP client retries by default.
+ public static final int HTTP_CLIENT_RETRIES = 0;
+
 
  private WaitUtil() {}
 
@@ -148,6 +152,7 @@ private boolean ping() throws IOException {
  .build();
  CloseableHttpClient httpClient = HttpClientBuilder.create()
  .setDefaultRequestConfig(requestConfig)
+ .setRetryHandler(new DefaultHttpRequestRetryHandler(HTTP_CLIENT_RETRIES, false))
  .build();
  try {
  CloseableHttpResponse response = httpClient.execute(RequestBuilder.create(method.toUpperCase()).setUri(url).build());