Skip to content

Commit 8423737

Browse files
committed
adjust ping intervals for --subscribe
1 parent 1d39cce commit 8423737

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

client/remote/client.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1141,7 +1141,7 @@ func (c *Client) pingLoop() {
11411141
defer c.logger.Debug("Ping loop terminated")
11421142

11431143
// Use longer ping interval in subscribe mode to accommodate hibernated connections
1144-
pingInterval := 5 * time.Second
1144+
pingInterval := 4 * time.Minute
11451145
if c.config.Subscribe {
11461146
pingInterval = 25 * time.Minute
11471147
c.logger.Info("Subscribe mode: using 25-minute ping interval for hibernated connections")
@@ -1174,10 +1174,7 @@ func (c *Client) pingLoop() {
11741174
}
11751175

11761176
// Use longer ping timeout in subscribe mode
1177-
pingTimeout := 10 * time.Second
1178-
if c.config.Subscribe {
1179-
pingTimeout = 30 * time.Second
1180-
}
1177+
pingTimeout := pingInterval * 2
11811178

11821179
err := conn.WriteControl(websocket.PingMessage, nil, time.Now().Add(pingTimeout))
11831180
if err != nil {

0 commit comments

Comments
 (0)