Skip to content

Conversation

@violetagg
Copy link
Member

Introduce PING frame support for proactive connection health monitoring on idle HTTP/2 connections. The feature sends PING frames to detect unresponsive peers and close stale connections.

Configuration via Http2SettingsSpec:

  • pingAckTimeout: Timeout for ACK response
  • pingAckDropThreshold: Maximum PING attempts (default: 1)

Requires idle timeout configuration:

  • Client: ConnectionProvider.maxIdleTime
  • Server: HttpServer.idleTimeout

The feature is opt-in and integrates automatically with connection pool eviction (client) and idle timeout handling (server).

This implementation is based on PR #3612, reduces the scope and adds HttpClient support.

@violetagg violetagg added this to the 1.2.12 milestone Oct 24, 2025
@violetagg violetagg added the type/enhancement A general enhancement label Oct 24, 2025
Introduce PING frame support for proactive connection health monitoring
on idle HTTP/2 connections. The feature sends PING frames to detect
unresponsive peers and close stale connections.

Configuration via Http2SettingsSpec:
- pingAckTimeout: Timeout for ACK response
- pingAckDropThreshold: Maximum PING attempts (default: 1)

Requires idle timeout configuration:
- Client: ConnectionProvider.maxIdleTime
- Server: HttpServer.idleTimeout

The feature is opt-in and integrates automatically with connection
pool eviction (client) and idle timeout handling (server).

This implementation is based on PR #3612, reduces the scope and adds HttpClient support.

Co-authored-by: raccoonback <kosb15@naver.com>

Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
Signed-off-by: Violeta Georgieva <696661+violetagg@users.noreply.github.com>
@raccoonback
Copy link
Contributor

@violetagg
I’ll review it by the end of this week! 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants