From 3463205b8c4f3c08d0fd3043bae8e92cac085f3e Mon Sep 17 00:00:00 2001
From: Corneil du Plessis <corneil.du-plessis@broadcom.com>
Date: Tue, 5 Nov 2024 09:55:05 +0200
Subject: [PATCH] Apply connection timeout correctly

This commit updates HttpComponents5MessageSender to apply the
configured connection timeout. Previously, it was wrongly set as the
connection request timeout.

Closes gh-1436
---
 .../ws/transport/http/HttpComponents5ClientFactory.java        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/spring-ws-core/src/main/java/org/springframework/ws/transport/http/HttpComponents5ClientFactory.java b/spring-ws-core/src/main/java/org/springframework/ws/transport/http/HttpComponents5ClientFactory.java
index b5ff4e19..d156b53b 100644
--- a/spring-ws-core/src/main/java/org/springframework/ws/transport/http/HttpComponents5ClientFactory.java
+++ b/spring-ws-core/src/main/java/org/springframework/ws/transport/http/HttpComponents5ClientFactory.java
@@ -200,6 +200,7 @@ public class HttpComponents5ClientFactory implements FactoryBean<CloseableHttpCl
 	}
 
 	@Override
+	@SuppressWarnings("deprecation")
 	public CloseableHttpClient getObject() throws Exception {
 
 		PoolingHttpClientConnectionManagerBuilder connectionManagerBuilder = PoolingHttpClientConnectionManagerBuilder
@@ -218,7 +219,7 @@ public class HttpComponents5ClientFactory implements FactoryBean<CloseableHttpCl
 		applyMaxConnectionsPerHost(this.connectionManager);
 
 		RequestConfig.Builder requestConfigBuilder = RequestConfig.custom() //
-			.setConnectionRequestTimeout(Timeout.of(this.connectionTimeout)) //
+			.setConnectTimeout(Timeout.of(this.connectionTimeout)) //
 			.setResponseTimeout(Timeout.of(this.readTimeout));
 
 		HttpClientBuilder httpClientBuilder = HttpClientBuilder.create() //