From bdc409eb1446388a86dddde607e188c38c1da2d5 Mon Sep 17 00:00:00 2001
From: Martin Redolatti <martin.redolatti@split.io>
Date: Fri, 29 Mar 2024 13:29:39 -0300
Subject: [PATCH 1/2] close http client

---
 .../java/io/split/engine/common/SplitAPI.java  | 18 +++++++++++++-----
 .../java/io/split/service/SplitHttpClient.java |  4 +++-
 .../io/split/service/SplitHttpClientImpl.java  |  5 +++++
 3 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/client/src/main/java/io/split/engine/common/SplitAPI.java b/client/src/main/java/io/split/engine/common/SplitAPI.java
index 229f34cb..adb0500d 100644
--- a/client/src/main/java/io/split/engine/common/SplitAPI.java
+++ b/client/src/main/java/io/split/engine/common/SplitAPI.java
@@ -19,7 +19,8 @@ private SplitAPI(SplitHttpClient httpClient, CloseableHttpClient sseHttpClient,
         _requestDecorator = requestDecorator;
     }
 
-    public static SplitAPI build(SplitHttpClient httpClient, CloseableHttpClient sseHttpClient, RequestDecorator requestDecorator){
+    public static SplitAPI build(SplitHttpClient httpClient, CloseableHttpClient sseHttpClient,
+            RequestDecorator requestDecorator) {
         return new SplitAPI(httpClient, sseHttpClient, requestDecorator);
     }
 
@@ -31,13 +32,20 @@ public CloseableHttpClient getSseHttpClient() {
         return _sseHttpClient;
     }
 
-    public RequestDecorator getRequestDecorator() { return  _requestDecorator; }
+    public RequestDecorator getRequestDecorator() {
+        return _requestDecorator;
+    }
 
-    public void close(){
+    public void close() {
+        try {
+            _httpClient.close();
+        } catch (Exception e) {
+            _log.error("Error trying to close regular http client", e);
+        }
         try {
             _sseHttpClient.close();
-        } catch (Exception e){
+        } catch (Exception e) {
             _log.error("Error trying to close sseHttpClient", e);
         }
     }
-}
\ No newline at end of file
+}
diff --git a/client/src/main/java/io/split/service/SplitHttpClient.java b/client/src/main/java/io/split/service/SplitHttpClient.java
index 1e44a49b..1c88bcd4 100644
--- a/client/src/main/java/io/split/service/SplitHttpClient.java
+++ b/client/src/main/java/io/split/service/SplitHttpClient.java
@@ -4,12 +4,14 @@
 import io.split.client.dtos.SplitHttpResponse;
 
 import org.apache.hc.core5.http.HttpEntity;
+
+import java.io.Closeable;
 import java.io.IOException;
 import java.net.URI;
 import java.util.List;
 import java.util.Map;
 
-public interface SplitHttpClient {
+public interface SplitHttpClient extends Closeable {
     /**
      * Wrapper for HTTP get method
      * 
diff --git a/client/src/main/java/io/split/service/SplitHttpClientImpl.java b/client/src/main/java/io/split/service/SplitHttpClientImpl.java
index ed4bb5aa..64ca3a55 100644
--- a/client/src/main/java/io/split/service/SplitHttpClientImpl.java
+++ b/client/src/main/java/io/split/service/SplitHttpClientImpl.java
@@ -140,4 +140,9 @@ private void setBasicHeaders(HttpRequest request) {
                 ? _apikey.substring(_apikey.length() - 4)
                 : _apikey);
     }
+
+    @Override
+    public void close() throws IOException {
+        _client.close();
+    }
 }

From c917ff98b3fd2f61dd54abd7201ae223eb394d1f Mon Sep 17 00:00:00 2001
From: Martin Redolatti <martin.redolatti@split.io>
Date: Fri, 29 Mar 2024 13:32:44 -0300
Subject: [PATCH 2/2] bump version in pom.xml files

---
 client/pom.xml            | 2 +-
 pluggable-storage/pom.xml | 2 +-
 pom.xml                   | 2 +-
 redis-wrapper/pom.xml     | 2 +-
 testing/pom.xml           | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/client/pom.xml b/client/pom.xml
index 12cda4f3..7a6a53ca 100644
--- a/client/pom.xml
+++ b/client/pom.xml
@@ -5,7 +5,7 @@
     <parent>
         <groupId>io.split.client</groupId>
         <artifactId>java-client-parent</artifactId>
-        <version>4.12.0-rc2</version>
+        <version>4.12.0-rc3</version>
     </parent>
     <artifactId>java-client</artifactId>
     <packaging>jar</packaging>
diff --git a/pluggable-storage/pom.xml b/pluggable-storage/pom.xml
index d7262066..d2d75905 100644
--- a/pluggable-storage/pom.xml
+++ b/pluggable-storage/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <artifactId>java-client-parent</artifactId>
         <groupId>io.split.client</groupId>
-        <version>4.12.0-rc2</version>
+        <version>4.12.0-rc3</version>
     </parent>
 
     <version>2.1.0</version>
diff --git a/pom.xml b/pom.xml
index 06f94991..91f89a44 100644
--- a/pom.xml
+++ b/pom.xml
@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>io.split.client</groupId>
     <artifactId>java-client-parent</artifactId>
-    <version>4.12.0-rc2</version>
+    <version>4.12.0-rc3</version>
     <dependencyManagement>
         <dependencies>
             <dependency>
diff --git a/redis-wrapper/pom.xml b/redis-wrapper/pom.xml
index 498ec361..0beb3ed7 100644
--- a/redis-wrapper/pom.xml
+++ b/redis-wrapper/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <artifactId>java-client-parent</artifactId>
         <groupId>io.split.client</groupId>
-        <version>4.12.0-rc2</version>
+        <version>4.12.0-rc3</version>
     </parent>
     <artifactId>redis-wrapper</artifactId>
     <version>3.1.0</version>
diff --git a/testing/pom.xml b/testing/pom.xml
index 4bbe4b31..e49a96c6 100644
--- a/testing/pom.xml
+++ b/testing/pom.xml
@@ -5,7 +5,7 @@
     <parent>
         <groupId>io.split.client</groupId>
         <artifactId>java-client-parent</artifactId>
-        <version>4.12.0-rc2</version>
+        <version>4.12.0-rc3</version>
     </parent>
     <artifactId>java-client-testing</artifactId>
     <packaging>jar</packaging>