feign icon indicating copy to clipboard operation
feign copied to clipboard

Re-enable Http2ClientReconnectTest - Need assistance from contributors

Open velo opened this issue 5 months ago • 0 comments

Summary

This PR reverts the temporary disabling of Http2ClientReconnectTest in both feign-vertx and feign-vertx4-test modules.

The test was temporarily disabled in commit 196f282 to prevent build failures, but it should be properly fixed rather than left disabled.

Need for Assistance

@aklenin (Alexei KLENIN) - Could you please help investigate and fix the issues with the HTTP/2 reconnection tests? Based on the git history, you both have significant experience with this test code:

  • Alexei implemented the original WebClient-based architecture in #2756
  • Marvin has made several updates to the test infrastructure

Test Failures

The Http2ClientReconnectTest is currently experiencing issues that cause build failures. The test extends AbstractClientReconnectTest and specifically tests HTTP/2 reconnection scenarios with:

  • Vertx WebClient with HTTP/2 protocol
  • Connection pool size limits
  • Jackson encoder/decoder integration

Action Items

  • [ ] Investigate root cause of test failures
  • [ ] Fix underlying issues with HTTP/2 reconnection logic
  • [ ] Ensure tests pass consistently in CI
  • [ ] Re-enable the tests

Any assistance in debugging and fixing these tests would be greatly appreciated!

🤖 Generated with Claude Code

velo avatar Jul 20 '25 23:07 velo