Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

fix unnecessary threadpool-creation #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix unnecessary threadpool-creation #40

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 2 additions & 1 deletion java/src/main/java/org/whispersystems/signalservice/internal/push/PushServiceSocket.java
View file
Open in desktop
Expand Up @@ -77,6 +77,7 @@
public class PushServiceSocket {

private static final String TAG = PushServiceSocket.class.getSimpleName();
private static final OkHttpClient CLIENT = new OkHttpClient();

private static final String CREATE_ACCOUNT_SMS_PATH = "/v1/accounts/sms/code/%s";
private static final String CREATE_ACCOUNT_VOICE_PATH = "/v1/accounts/voice/code/%s";
Expand Down Expand Up @@ -612,7 +613,7 @@ private Response getConnection(String urlFragment, String method, String body)
SSLContext context = SSLContext.getInstance("TLS");
context.init(null, trustManagers, null);

OkHttpClient.Builder okHttpClientBuilder = new OkHttpClient.Builder()
OkHttpClient.Builder okHttpClientBuilder = CLIENT.newBuilder()
.sslSocketFactory(context.getSocketFactory(), (X509TrustManager)trustManagers[0])
.connectTimeout(soTimeoutMillis, TimeUnit.MILLISECONDS)
.readTimeout(soTimeoutMillis, TimeUnit.MILLISECONDS);
Expand Down