From f5bc56198d65a3a3312439a39e9d963b2b016503 Mon Sep 17 00:00:00 2001 From: Ryan Schmitt Date: Sun, 6 Jul 2025 12:10:15 -0700 Subject: [PATCH] TestValidateAfterInactivity: Set maxConns to 1 The test server only supports a single connection at a time, and the async client can apparently attempt to establish a second connection, possibly due to a benign race condition in how leased connections are returned to the pool. --- .../apache/hc/client5/testing/TestValidateAfterInactivity.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/httpclient5-testing/src/test/java/org/apache/hc/client5/testing/TestValidateAfterInactivity.java b/httpclient5-testing/src/test/java/org/apache/hc/client5/testing/TestValidateAfterInactivity.java index e40eb6771f..47ccac911e 100644 --- a/httpclient5-testing/src/test/java/org/apache/hc/client5/testing/TestValidateAfterInactivity.java +++ b/httpclient5-testing/src/test/java/org/apache/hc/client5/testing/TestValidateAfterInactivity.java @@ -260,6 +260,8 @@ private CloseableHttpClient syncClient(final boolean validateAfterInactivity) { private CloseableHttpAsyncClient asyncClient(final boolean validateAfterInactivity) { final PoolingAsyncClientConnectionManager connManager = new PoolingAsyncClientConnectionManager(); connManager.setDefaultConnectionConfig(getConnectionConfig(validateAfterInactivity)); + connManager.setDefaultMaxPerRoute(1); + connManager.setMaxTotal(1); final CloseableHttpAsyncClient client = HttpAsyncClients.custom() .setConnectionManager(connManager) .disableAutomaticRetries()