fix: use http1.1 by default (#148)

* fix: use http1.1 by default

* fmt fix

Author: jimmyjames

src/main/java/dev/openfga/sdk/api/client/ApiClient.java

@@ -67,6 +67,24 @@ public ApiClient() {
         asyncResponseInterceptor = null;
     }
 
+    /**
+     * Create an instance of ApiClient.
+     * <p>
+     * In other contexts, note that any settings in a {@link Configuration}
+     * will take precedence over equivalent settings in the
+     * {@link HttpClient.Builder} here.
+     *
+     * @param builder Http client builder.
+     */
+    public ApiClient(HttpClient.Builder builder) {
+        this.builder = builder;
+        this.mapper = createDefaultObjectMapper();
+        this.client = this.builder.build();
+        interceptor = null;
+        responseInterceptor = null;
+        asyncResponseInterceptor = null;
+    }
+
     /**
      * Create an instance of ApiClient.
      * <p>
@@ -177,7 +195,7 @@ protected String getDefaultBaseUri() {
     }
 
     protected HttpClient.Builder createDefaultHttpClientBuilder() {
-        return HttpClient.newBuilder();
+        return HttpClient.newBuilder().version(HttpClient.Version.HTTP_1_1);
     }
 
     /**

src/test/java/dev/openfga/sdk/api/client/ApiClientTest.java

@@ -35,4 +35,18 @@ public void newHttpClientWhenBuilderModified() {
 
         assertNotEquals(client1, apiClient.getHttpClient());
     }
+
+    @Test
+    public void httpClientShouldUseHttp1ByDefault() {
+        ApiClient apiClient = new ApiClient();
+        assertEquals(apiClient.getHttpClient().version(), HttpClient.Version.HTTP_1_1);
+    }
+
+    @Test
+    public void customHttpClientWithHttp2() {
+        HttpClient.Builder builder = HttpClient.newBuilder().version(HttpClient.Version.HTTP_2);
+        ApiClient apiClient = new ApiClient(builder);
+        ;
+        assertEquals(apiClient.getHttpClient().version(), HttpClient.Version.HTTP_2);
+    }
 }