Skip to content

Commit 75a1678

Browse files
veloclaude
andcommitted
Fix HTTP/2 client reconnect test timing issues for CI
- Add longer connection and idle timeouts for HTTP/2 client - Add small delays between requests to help with CI timing - Disable HTTP/2 clear text upgrade to avoid connection issues 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 28df6b0 commit 75a1678

2 files changed

Lines changed: 10 additions & 2 deletions

File tree

vertx/feign-vertx4-test/src/test/java/feign/vertx/AbstractClientReconnectTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,10 @@ void allRequestsShouldBeAnswered(VertxTestContext testContext) {
131131
CompositeFuture sendRequests(int requests) {
132132
List<Future<?>> requestList =
133133
IntStream.range(0, requests)
134-
.mapToObj(ignored -> client.hello())
134+
.mapToObj(i -> {
135+
// Add small delay between requests to help with CI timing
136+
return Future.timer(vertx, i * 50).compose(ignore -> client.hello());
137+
})
135138
.collect(Collectors.toList());
136139
return Future.all(requestList);
137140
}

vertx/feign-vertx4-test/src/test/java/feign/vertx/Http2ClientReconnectTest.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,12 @@ class Http2ClientReconnectTest extends AbstractClientReconnectTest {
3333
@Override
3434
protected void createClient(Vertx vertx) {
3535
WebClientOptions options =
36-
new WebClientOptions().setProtocolVersion(HttpVersion.HTTP_2).setHttp2MaxPoolSize(1);
36+
new WebClientOptions()
37+
.setProtocolVersion(HttpVersion.HTTP_2)
38+
.setHttp2MaxPoolSize(1)
39+
.setConnectTimeout(10000)
40+
.setIdleTimeout(30000)
41+
.setHttp2ClearTextUpgrade(false);
3742
WebClient webClient = WebClient.create(vertx, options);
3843

3944
client =

0 commit comments

Comments
 (0)