[java] Stopping OkHttp from retrying after getting 408 response code. This caused duplication of page load timeout.

Author: barancev

java/client/src/org/openqa/selenium/remote/internal/OkHttpClient.java

@@ -142,6 +142,7 @@ public HttpClient createClient(URL url) {
           .connectionPool(pool)
           .followRedirects(true)
           .followSslRedirects(true)
+          .retryOnConnectionFailure(false)
           .readTimeout(readTimeout, MILLISECONDS)
           .connectTimeout(connectionTimeout, MILLISECONDS);
 

java/client/test/org/openqa/selenium/PageLoadingTest.java

@@ -461,7 +461,7 @@ public void testShouldTimeoutIfAPageTakesTooLongToRefresh() {
 
       int duration = (int) (end - start);
       assertThat(duration, greaterThan(2000));
-      assertThat(duration, lessThan(5000));
+      assertThat(duration, lessThan(4000));
     } finally {
       driver.manage().timeouts().pageLoadTimeout(300, SECONDS);
     }