Consistently use the same OkHttpClient for files.slack.com calls (#1428)

slackapi · Jan 28, 2025 · 0764bd8 · 0764bd8
1 parent 2094cf2
commit 0764bd8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/slack-api-client/src/main/java/com/slack/api/methods/impl/FilesUploadV2Helper.java b/slack-api-client/src/main/java/com/slack/api/methods/impl/FilesUploadV2Helper.java
@@ -31,7 +31,7 @@ public class FilesUploadV2Helper implements AutoCloseable {
 
     public FilesUploadV2Helper(MethodsClient client) {
         this.client = client;
-        this.okHttpClient = SlackHttpClient.buildOkHttpClient(client.getSlackHttpClient().getConfig());
+        this.okHttpClient = client.getSlackHttpClient().getOkHttpClient();
     }
 
     public String uploadFile(

diff --git a/slack-api-client/src/main/java/com/slack/api/util/http/SlackHttpClient.java b/slack-api-client/src/main/java/com/slack/api/util/http/SlackHttpClient.java
@@ -123,6 +123,10 @@ public void setConfig(SlackConfig config) {
         this.config = config;
     }
 
+    public OkHttpClient getOkHttpClient() {
+        return this.okHttpClient;
+    }
+
     public Response get(String url, Map<String, String> query, String token) throws IOException {
         if (query != null) {
             HttpUrl.Builder u = HttpUrl.parse(url).newBuilder();