Keepalive bugfixes and unify timers strategies between client and server#2760

Merged

murgatroid99 merged 12 commits intogrpc:@grpc/[email protected]from

davidfiala:@grpc/[email protected]

Jun 18, 2024

Commits on May 28, 2024

Commits on May 29, 2024

unify server and client keepalive matching comments and discussion on first round of review from https://github.com/grpc/grpc-node/pull/2760
davidfiala
committed
add keepalive server trace back in to match channelz vs non-channelz trace behavior
davidfiala
committed
return imports back to original order
davidfiala
committed
another missing trace message for parity
davidfiala
committed
resolve hoisting
davidfiala
committed
hoist in second location
davidfiala
committed
Based on grpc/grpc-node#2139 I wrapped http2session.ping in a try-catch block again
davidfiala
committed
remove keepaliveDisabled from server.ts. rename keepaliveTimer.
davidfiala
committed

Commits on Jun 6, 2024

per discussion, avoid tracking keepalive disabled state and instead depend on whether the session is destroyed
davidfiala
committed

Commits on Jun 7, 2024

ensure that client keepalive timers are always cleared when they trigger. this is a necessary change to fit with having removed keepaliveDisabled boolean. manually inspected test logs for both serv…
davidfiala
authored