KeepAlive client
coot opened this issue · comments
Marcin Szamotulski commented
The keep alive client has two small issues:
- it should record time when it sends the initial
MsgKeepAlive
, and compute thertt
when we receive the reply. - currently after first round trip with
MsgKeepAlive -> MsgKeepAliveResponse
, we immediately send the nextMsgKeepAlive
message without doing the delay. If we refactor the code to fix the first issue we will actually fix this one as well, if we eliminate ref.
Karl Knutsson commented
Will look into this.