fix(sdk): kasclient no longer closes shared http client

cshamrick · cshamrick · commit 88f7520938da · 2025-12-08T17:54:39.000-06:00
Signed-off-by: Scott Hamrick &lt;2623452+cshamrick@users.noreply.github.com&gt;
diff --git a/sdk/src/main/java/io/opentdf/platform/sdk/KASClient.java b/sdk/src/main/java/io/opentdf/platform/sdk/KASClient.java
@@ -125,8 +125,7 @@ public KASKeyCache getKeyCache() {
 
     @Override
     public synchronized void close() {
-        this.httpClient.dispatcher().cancelAll();
-        this.httpClient.connectionPool().evictAll();
+        // KASClient is no longer responsible for closing the shared http client
     }
 
     static class RewrapRequestBody {

Original file line number	Diff line number	Diff line change
`@@ -125,8 +125,7 @@ public KASKeyCache getKeyCache() {`
`125`	`125`
`126`	`126`	`@Override`
`127`	`127`	`public synchronized void close() {`
`128`		`- this.httpClient.dispatcher().cancelAll();`
`129`		`- this.httpClient.connectionPool().evictAll();`
	`128`	`+ // KASClient is no longer responsible for closing the shared http client`
`130`	`129`	`}`
`131`	`130`
`132`	`131`	`static class RewrapRequestBody {`