Skip to content

Commit 62cfd23

Browse files
committed
Deprecate getOkHttpClientBuilder and make createNewClientBuilder public.
1 parent 53830af commit 62cfd23

2 files changed

Lines changed: 25 additions & 12 deletions

File tree

libs/SalesforceSDK/src/com/salesforce/androidsdk/auth/HttpAccess.java

Lines changed: 24 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -98,28 +98,41 @@ public HttpAccess(Context app, String userAgent) {
9898
*
9999
* @return okHttpClient.Builder with appropriate connection spec
100100
* and user agent interceptor for an authenticated client.
101+
*
102+
* @deprecated To be removed in 14.0. Please use {@link #createNewClientBuilder()} instead.
101103
*/
104+
@Deprecated
102105
public OkHttpClient.Builder getOkHttpClientBuilder() {
103-
ConnectionSpec connectionSpec = new ConnectionSpec.Builder(ConnectionSpec.MODERN_TLS)
104-
.tlsVersions(TlsVersion.TLS_1_1, TlsVersion.TLS_1_2)
105-
.build();
106-
return new OkHttpClient.Builder()
107-
.connectionSpecs(Collections.singletonList(connectionSpec))
108-
.connectTimeout(CONNECT_TIMEOUT, TimeUnit.SECONDS)
109-
.readTimeout(READ_TIMEOUT, TimeUnit.SECONDS)
110-
.addNetworkInterceptor(new UserAgentInterceptor());
106+
return createNewClientBuilder();
111107
}
112108

113109
/**
114110
*
115111
* @return okHttpClient.Builder with appropriate connection spec
116112
* and user agent interceptor for an unauthenticated client.
117113
*
118-
* @deprecated To be removed in 14.0. Please use {{@link #getOkHttpClientBuilder()}} instead.
114+
* @deprecated To be removed in 14.0. Please use {@link #createNewClientBuilder()} instead.
119115
*/
120116
@Deprecated
121117
public OkHttpClient.Builder getUnauthenticatedOkHttpBuilder() {
122-
return getOkHttpClientBuilder();
118+
return createNewClientBuilder();
119+
}
120+
121+
/**
122+
* Creates a new OkHttp Client Builder with appropriate connection spec
123+
* and user agent interceptor.
124+
*
125+
* @return the okHttpClient.Builder
126+
*/
127+
public OkHttpClient.Builder createNewClientBuilder() {
128+
ConnectionSpec connectionSpec = new ConnectionSpec.Builder(ConnectionSpec.MODERN_TLS)
129+
.tlsVersions(TlsVersion.TLS_1_1, TlsVersion.TLS_1_2)
130+
.build();
131+
return new OkHttpClient.Builder()
132+
.connectionSpecs(Collections.singletonList(connectionSpec))
133+
.connectTimeout(CONNECT_TIMEOUT, TimeUnit.SECONDS)
134+
.readTimeout(READ_TIMEOUT, TimeUnit.SECONDS)
135+
.addNetworkInterceptor(new UserAgentInterceptor());
123136
}
124137

125138
/**
@@ -128,7 +141,7 @@ public OkHttpClient.Builder getUnauthenticatedOkHttpBuilder() {
128141
*/
129142
public synchronized OkHttpClient getOkHttpClient() {
130143
if (okHttpClient == null) {
131-
okHttpClient = getOkHttpClientBuilder().build();
144+
okHttpClient = createNewClientBuilder().build();
132145
}
133146
return okHttpClient;
134147
}

libs/SalesforceSDK/src/com/salesforce/androidsdk/rest/RestClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ private synchronized void setOkHttpClientBuilder() {
204204

205205
// If none cached, create new one
206206
if (okHttpClientBuilder == null) {
207-
okHttpClientBuilder = httpAccessor.getOkHttpClientBuilder();
207+
okHttpClientBuilder = httpAccessor.createNewClientBuilder();
208208
if (!cacheKey.equals("unauthenticated")) {
209209
okHttpClientBuilder.addInterceptor(getOAuthRefreshInterceptor());
210210
}

0 commit comments

Comments
 (0)