Add HTTP/2 PING for broken connection detection.#49095

Merged

jeet1995 merged 68 commits into

jeet1995:squad/http2-ping-keepalive

Jun 6, 2026

Commits on May 25, 2026

Extract HTTP/2 PING keepalive from PR #48420

jeet1995
and
Copilot
committed
Remove IHttpClientInterceptor plumbing, add PING concurrency guard + broken connection detection
jeet1995
committed
Align PING timing with Rust SDK: interval=1s, timeout=2s, close on timeout
jeet1995
committed
Add PING interval and timeout tests, remove thin client dependency
jeet1995
committed
Force single-connection H2 pool in tests for deterministic channel assertions
jeet1995
committed
Add Cosmos_Live_Test_Http2NetworkFault CI stage, delete unused Http2PingFrameCounterHandler
jeet1995
committed
Add consecutive failure threshold for defensive PING-based connection close
jeet1995
committed
Align failure threshold with Rust SDK (5), test overrides to 2 for speed
jeet1995
committed
Remove spec doc — design decisions documented in PR description
jeet1995
committed
Add sudo detection for CI VMs, PreSteps for iptables installation
jeet1995
committed
Log levels: WARN only for connection close, INFO for retries and send failures
jeet1995
committed
Remove unnecessary isDebugEnabled guards — SLF4J parameterized logging handles this
jeet1995
committed
Single INFO log for connection close, everything else DEBUG
jeet1995
committed
Use Http2ConnectionConfig API for pool size, replace em dashes with --
jeet1995
committed
Fix readAllBytes() compile error -- use BufferedReader for JDK 8 compat
jeet1995
committed
Deep review cleanup: remove unused Http2ConnectionConfig import, replace all em dashes
jeet1995
committed
Replace global counters with channel attribute for handler ref, assert connection identity only
jeet1995
committed
Add CHANGELOG entry for HTTP/2 PING keepalive
jeet1995
committed
Simplify CHANGELOG wording
jeet1995
committed
Remove unused PING_HANDLER_REF attribute and getFrom() method
jeet1995
committed
Address Copilot review comments
jeet1995
committed
Address xinlian12 review comments (4-8)
jeet1995
committed
Remove test 1 (pingFramesSentAtConfiguredInterval) -- does not prove PINGs
jeet1995
committed
Address review round 2: remove dead code, track lastReadNanos, bounds guard
jeet1995
committed
Add HTTP/2 PING resilience: probe PING and client-level kill switch

jeet1995
and
Copilot
committed
Self-review: move http2CfgAccessor to static field, remove write() override and active-streams check, add thread-safety comment
jeet1995
committed
Remove probe PING, remove active-streams check, revert channelInactive kill-switch; PROTOCOL_ERROR via exceptionCaught is the reliable path
jeet1995
committed
Use lazy init pattern for http2CfgAccessor
jeet1995
committed

Commits on May 27, 2026

Commits on Jun 2, 2026

Commits on Jun 3, 2026

Commits on Jun 4, 2026

Commits on Jun 5, 2026