Skip to content

Commit ee0ccc2

Browse files
chrisribblesenivam
authored andcommitted
jnh-connector: Respect ClientProperties.CONNECT_TIMEOUT
1 parent 77bb3e5 commit ee0ccc2

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

connectors/jnh-connector/src/main/java/org/glassfish/jersey/jnh/connector/JavaNetHttpConnector.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,11 @@ public JavaNetHttpConnector(final Client client, final Configuration configurati
9191
if (sslContext != null) {
9292
httpClientBuilder.sslContext(sslContext);
9393
}
94+
final Integer connectTimeout =
95+
getPropertyOrNull(configuration, ClientProperties.CONNECT_TIMEOUT, Integer.class);
96+
if (connectTimeout != null) {
97+
httpClientBuilder.connectTimeout(Duration.ofMillis(connectTimeout));
98+
}
9499
final CookieHandler cookieHandler =
95100
getPropertyOrNull(configuration, JavaNetHttpClientProperties.COOKIE_HANDLER, CookieHandler.class);
96101
if (cookieHandler != null) {
@@ -205,9 +210,9 @@ private HttpRequest getHttpRequest(ClientRequest request) {
205210
builder.header(headerName, headerValue);
206211
}
207212
}
208-
final Integer connectTimeout = request.resolveProperty(ClientProperties.READ_TIMEOUT, Integer.class);
209-
if (connectTimeout != null) {
210-
builder.timeout(Duration.ofMillis(connectTimeout));
213+
final Integer readTimeout = request.resolveProperty(ClientProperties.READ_TIMEOUT, Integer.class);
214+
if (readTimeout != null) {
215+
builder.timeout(Duration.ofMillis(readTimeout));
211216
}
212217
processExtensions(builder, request);
213218
return builder.build();

0 commit comments

Comments
 (0)