Skip to content

Commit cc3ba34

Browse files
committed
SCANJLIB-198 Update okhttp and other libraries
1 parent c6b6eb0 commit cc3ba34

3 files changed

Lines changed: 8 additions & 17 deletions

File tree

lib/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,6 @@
6161
<artifactId>mockito-core</artifactId>
6262
<scope>test</scope>
6363
</dependency>
64-
<dependency>
65-
<groupId>org.mockito</groupId>
66-
<artifactId>mockito-inline</artifactId>
67-
<scope>test</scope>
68-
</dependency>
6964
<dependency>
7065
<groupId>org.assertj</groupId>
7166
<artifactId>assertj-core</artifactId>

lib/src/main/java/org/sonarsource/scanner/lib/internal/OkHttpClientFactory.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import java.time.Duration;
3434
import java.util.Arrays;
3535
import java.util.concurrent.TimeUnit;
36+
import javax.annotation.Nullable;
3637
import javax.net.ssl.KeyManager;
3738
import javax.net.ssl.KeyManagerFactory;
3839
import javax.net.ssl.SSLContext;
@@ -42,8 +43,8 @@
4243
import javax.net.ssl.X509TrustManager;
4344
import okhttp3.ConnectionSpec;
4445
import okhttp3.Credentials;
45-
import okhttp3.OkHttpClient;
4646
import okhttp3.JavaNetCookieJar;
47+
import okhttp3.OkHttpClient;
4748
import org.sonarsource.scanner.lib.internal.cache.Logger;
4849

4950
import static java.util.Arrays.asList;
@@ -57,7 +58,8 @@ public class OkHttpClientFactory {
5758
static final String P11KEYSTORE = "PKCS11";
5859
static final CookieManager COOKIE_MANAGER;
5960
private static final String PROXY_AUTHORIZATION = "Proxy-Authorization";
60-
private static final JavaNetCookieJar COOKIE_JAR; // use the same cookie jar for all instances
61+
// use the same cookie jar for all instances
62+
private static final JavaNetCookieJar COOKIE_JAR;
6163

6264
private OkHttpClientFactory() {
6365
// only statics
@@ -84,7 +86,6 @@ static OkHttpClient create(Logger logger) {
8486
ConnectionSpec tls = new ConnectionSpec.Builder(ConnectionSpec.MODERN_TLS)
8587
.allEnabledTlsVersions()
8688
.allEnabledCipherSuites()
87-
.supportsTlsExtensions(true)
8889
.build();
8990
okHttpClientBuilder.connectionSpecs(asList(tls, ConnectionSpec.CLEARTEXT));
9091

@@ -183,7 +184,7 @@ private static synchronized KeyManager[] getDefaultKeyManager(Logger logger) {
183184
}
184185
}
185186

186-
private static KeyStore initKeyStore(Logger logger, final String defaultKeyStore, String defaultKeyStoreType, String defaultKeyStoreProvider, char[] passwd)
187+
private static KeyStore initKeyStore(Logger logger, final String defaultKeyStore, String defaultKeyStoreType, String defaultKeyStoreProvider, @Nullable char[] passwd)
187188
throws KeyStoreException, NoSuchProviderException, IOException, NoSuchAlgorithmException, CertificateException {
188189
KeyStore ks = null;
189190
if (!defaultKeyStoreType.isEmpty()) {

pom.xml

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353

5454
<!-- used for deployment to SonarSource Artifactory -->
5555
<gitRepositoryName>sonar-scanner-java-library</gitRepositoryName>
56-
<okhttp.version>3.14.2</okhttp.version>
56+
<okhttp.version>4.10.0</okhttp.version>
5757
<mockito.version>3.12.4</mockito.version>
5858
</properties>
5959

@@ -67,7 +67,7 @@
6767
<dependency>
6868
<groupId>commons-io</groupId>
6969
<artifactId>commons-io</artifactId>
70-
<version>2.8.0</version>
70+
<version>2.15.1</version>
7171
</dependency>
7272
<dependency>
7373
<groupId>com.squareup.okhttp3</groupId>
@@ -109,11 +109,6 @@
109109
<artifactId>mockito-core</artifactId>
110110
<version>${mockito.version}</version>
111111
</dependency>
112-
<dependency>
113-
<groupId>org.mockito</groupId>
114-
<artifactId>mockito-inline</artifactId>
115-
<version>${mockito.version}</version>
116-
</dependency>
117112
<dependency>
118113
<groupId>org.assertj</groupId>
119114
<artifactId>assertj-core</artifactId>
@@ -122,7 +117,7 @@
122117
<dependency>
123118
<groupId>commons-codec</groupId>
124119
<artifactId>commons-codec</artifactId>
125-
<version>1.11</version>
120+
<version>1.16.1</version>
126121
</dependency>
127122
</dependencies>
128123
</dependencyManagement>

0 commit comments

Comments
 (0)