Merge pull request #131 from okta/ak_bump_okhttp3

upgrade okhttp3 and auto-service deps

Author: arvindkrishnakumar-okta

THIRD-PARTY-NOTICES

@@ -28,13 +28,13 @@ This project includes:
   Guava ListenableFuture only under The Apache Software License, Version 2.0
   Guava: Google Core Libraries for Java under Apache License, Version 2.0
   IntelliJ IDEA Annotations under The Apache Software License, Version 2.0
-  J2ObjC Annotations under The Apache Software License, Version 2.0
+  J2ObjC Annotations under Apache License, Version 2.0
   Kotlin Stdlib under The Apache License, Version 2.0
   Kotlin Stdlib Common under The Apache License, Version 2.0
   Kotlin Stdlib Jdk7 under The Apache License, Version 2.0
   Kotlin Stdlib Jdk8 under The Apache License, Version 2.0
   okhttp under The Apache Software License, Version 2.0
-  OkHttp Logging Interceptor under Apache 2.0
+  okhttp-logging-interceptor under The Apache Software License, Version 2.0
   okio under The Apache Software License, Version 2.0
 
 

http/okhttp/pom.xml

@@ -49,13 +49,13 @@
         <dependency>
             <groupId>com.squareup.okhttp3</groupId>
             <artifactId>okhttp</artifactId>
-            <version>4.10.0</version>
+            <version>4.11.0</version>
         </dependency>
 
         <dependency>
             <groupId>com.squareup.okhttp3</groupId>
             <artifactId>logging-interceptor</artifactId>
-            <version>3.14.4</version>
+            <version>4.10.0</version>
         </dependency>
 
         <dependency>

http/okhttp/src/main/java/com/okta/commons/http/okhttp/OkHttpRequestExecutor.java

@@ -104,7 +104,7 @@ static OkHttpClient configureOkHttpClient(HttpClientConfiguration httpClientConf
         clientBuilder.writeTimeout(httpClientConfiguration.getConnectionTimeout(), TimeUnit.SECONDS);
 
         clientBuilder.cookieJar(CookieJar.NO_COOKIES);
-        clientBuilder.retryOnConnectionFailure(false); // handled by SDK
+        clientBuilder.retryOnConnectionFailure(true); // fix for https://github.com/square/okhttp/issues/2738
 
         final Proxy sdkProxy = httpClientConfiguration.getProxy();
         if (sdkProxy != null) {

http/okhttp/src/test/groovy/com/okta/commons/http/okhttp/OkHttpRequestExecutorFactoryTest.groovy

@@ -69,7 +69,7 @@ class OkHttpRequestExecutorFactoryTest {
         assertThat okHttpClient.cookieJar(), is(cookieJar)
         assertThat client.cookieJar(), is(CookieJar.NO_COOKIES)
         assertThat okHttpClient.retryOnConnectionFailure(), is(true)
-        assertThat client.retryOnConnectionFailure(), is(false)
+        assertThat client.retryOnConnectionFailure(), is(true)
     }
 
     private static OkHttpClient okHttpClientFromExecutor(RequestExecutor executor) {

pom.xml

@@ -92,14 +92,14 @@
             <dependency>
                 <groupId>com.squareup.okio</groupId>
                 <artifactId>okio-jvm</artifactId>
-                <version>3.2.0</version>
+                <version>3.4.0</version>
             </dependency>
 
             <dependency>
                 <!-- compile time annotation processor -->
                 <groupId>com.google.auto.service</groupId>
                 <artifactId>auto-service</artifactId>
-                <version>1.0.1</version>
+                <version>1.1.1</version>
                 <optional>true</optional>
             </dependency>