Skip to content

Commit a907644

Browse files
Switch to JDK HTTP Client
Fixes #480
1 parent 7482816 commit a907644

10 files changed

Lines changed: 373 additions & 371 deletions

File tree

pom.xml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
<project.build.reportEncoding>UTF-8</project.build.reportEncoding>
1818

1919
<!-- Project dependency versions -->
20-
<httpcomponents.version>5.6</httpcomponents.version>
2120
<jackson.version>2.21.1</jackson.version>
2221

2322
<!-- Project test dependency versions -->
@@ -91,12 +90,6 @@
9190
<groupId>com.fasterxml.jackson.datatype</groupId>
9291
<artifactId>jackson-datatype-jsr310</artifactId>
9392
</dependency>
94-
<dependency>
95-
<groupId>org.apache.httpcomponents.client5</groupId>
96-
<artifactId>httpclient5</artifactId>
97-
<version>${httpcomponents.version}</version>
98-
</dependency>
99-
10093
<dependency>
10194
<groupId>org.junit.jupiter</groupId>
10295
<artifactId>junit-jupiter</artifactId>

src/main/java/com/github/jamesnetherton/zulip/client/http/ZulipConfiguration.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package com.github.jamesnetherton.zulip.client.http;
22

3-
import com.github.jamesnetherton.zulip.client.http.commons.ZulipCommonsHttpClientFactory;
3+
import com.github.jamesnetherton.zulip.client.http.jdk.ZulipJdkHttpClientFactory;
44
import com.github.jamesnetherton.zulip.client.util.ZulipUrlUtils;
55
import java.io.File;
66
import java.io.IOException;
@@ -21,7 +21,7 @@ public class ZulipConfiguration {
2121
private URL proxyUrl;
2222
private String proxyUsername;
2323
private String proxyPassword;
24-
private ZulipHttpClientFactory zulipHttpClientFactory = new ZulipCommonsHttpClientFactory();
24+
private ZulipHttpClientFactory zulipHttpClientFactory = new ZulipJdkHttpClientFactory();
2525
private URL zulipUrl;
2626

2727
/**

src/main/java/com/github/jamesnetherton/zulip/client/http/ZulipHttpClientFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
/**
66
* Factory class for creating {@link ZulipHttpClient} implementations.
77
*
8-
* Provides a way to use HTTP client libraries other than the default Apache Commons HTTP Client.
8+
* Provides a way to use HTTP client libraries other than the default JDK HTTP Client.
99
*/
1010
public interface ZulipHttpClientFactory {
1111
ZulipHttpClient createZulipHttpClient(ZulipConfiguration configuration) throws ZulipClientException;

src/main/java/com/github/jamesnetherton/zulip/client/http/commons/ZulipCommonsHttpClient.java

Lines changed: 0 additions & 338 deletions
This file was deleted.

0 commit comments

Comments
 (0)