Skip to content

Commit a0faa92

Browse files
authored
SCANNERAPI-181 Update OkHttp to 3.14.0
1 parent 6941cb8 commit a0faa92

5 files changed

Lines changed: 3 additions & 32 deletions

File tree

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@
4040
import okhttp3.ConnectionSpec;
4141
import okhttp3.Credentials;
4242
import okhttp3.OkHttpClient;
43-
import okhttp3.internal.tls.OkHostnameVerifier;
4443
import org.sonarsource.scanner.api.internal.cache.Logger;
4544

4645
import static java.util.Arrays.asList;
@@ -72,7 +71,6 @@ static OkHttpClient create(Logger logger) {
7271

7372
X509TrustManager systemDefaultTrustManager = systemDefaultTrustManager();
7473
okHttpClientBuilder.sslSocketFactory(systemDefaultSslSocketFactory(systemDefaultTrustManager, logger), systemDefaultTrustManager);
75-
okHttpClientBuilder.hostnameVerifier(OkHostnameVerifier.INSTANCE);
7674

7775
// OkHttp detect 'http.proxyHost' java property, but credentials should be filled
7876
final String proxyUser = System.getProperty("http.proxyUser", "");

batch/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,6 @@
4141
</exclusions>
4242
</dependency>
4343
<dependency>
44-
<groupId>org.easytesting</groupId>
45-
<artifactId>fest-assert</artifactId>
46-
<scope>test</scope>
47-
</dependency>
48-
<dependency>
4944
<groupId>junit</groupId>
5045
<artifactId>junit</artifactId>
5146
<scope>test</scope>

batch/src/test/java/org/sonarsource/scanner/api/internal/batch/DefaultBatchFactoryTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
import org.junit.Test;
2525
import org.sonar.batch.bootstrapper.Batch;
2626

27-
import static org.fest.assertions.Assertions.assertThat;
27+
import static org.assertj.core.api.Assertions.assertThat;
2828

2929
public class DefaultBatchFactoryTest {
3030

its/it-tests/pom.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@
5757
<dependency>
5858
<groupId>org.assertj</groupId>
5959
<artifactId>assertj-core</artifactId>
60-
<version>2.3.0</version>
6160
<scope>test</scope>
6261
</dependency>
6362

pom.xml

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@
5252

5353
<!-- used for deployment to SonarSource Artifactory -->
5454
<gitRepositoryName>sonar-scanner-api</gitRepositoryName>
55-
<okhttp.version>3.11.0</okhttp.version>
56-
<mockito.version>2.22.0</mockito.version>
55+
<okhttp.version>3.14.0</okhttp.version>
56+
<mockito.version>2.23.4</mockito.version>
5757
</properties>
5858

5959
<profiles>
@@ -97,11 +97,6 @@
9797
<artifactId>commons-io</artifactId>
9898
<version>2.6</version>
9999
</dependency>
100-
<dependency>
101-
<groupId>org.easytesting</groupId>
102-
<artifactId>fest-assert</artifactId>
103-
<version>1.4</version>
104-
</dependency>
105100
<dependency>
106101
<groupId>com.squareup.okhttp3</groupId>
107102
<artifactId>okhttp</artifactId>
@@ -150,20 +145,4 @@
150145
</dependencies>
151146
</dependencyManagement>
152147

153-
<build>
154-
<pluginManagement>
155-
<plugins>
156-
<plugin>
157-
<groupId>org.apache.maven.plugins</groupId>
158-
<artifactId>maven-dependency-plugin</artifactId>
159-
<version>2.10</version>
160-
</plugin>
161-
<plugin>
162-
<groupId>org.apache.maven.plugins</groupId>
163-
<artifactId>maven-shade-plugin</artifactId>
164-
<version>3.0.0</version>
165-
</plugin>
166-
</plugins>
167-
</pluginManagement>
168-
</build>
169148
</project>

0 commit comments

Comments
 (0)