yamux: disable yamux keep alive in server channel

yamux client runs in the proxy side, sometimes the client is handling other requests and it's not able to response to the ping sent by the server and the communication is closed. To avoid IO timeouts in the communication between agent and proxy, keep alive should be disabled. fixes kata-containers/proxy#70 fixes #231 Signed-off-by: Julio Montes <[email protected]>
kata-containers · Jun 18, 2018 · 83d7906 · 83d7906
1 parent b47b313
commit 83d7906
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/channel.go b/channel.go
@@ -144,8 +144,14 @@ func (c *serialChannel) wait() error {
 }
 
 func (c *serialChannel) listen() (net.Listener, error) {
+	config := yamux.DefaultConfig()
+	// yamux client runs in the proxy side, sometimes the client is handling other requests and it's not able
+	// to response to the ping sent by the server and the communication is closed. To avoid IO timeouts in
+	// the communication between agent and proxy, keep alive should be disabled.
+	config.EnableKeepAlive = false
+
 	// Initialize Yamux server.
-	session, err := yamux.Server(c.serialConn, nil)
+	session, err := yamux.Server(c.serialConn, config)
 	if err != nil {
 		return nil, err
 	}