Disable keep-alive for SSH tunnel health checks

We don't reuse the http client. Disabling the keep-alive to properly close the connections.

Disable keep-alive for SSH tunnel health checks
e105eec9 · Yu-Ju Hong · c41c24fb · e105eec9
Commit e105eec9 authored Nov 08, 2016 by Yu-Ju Hong
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

ssh.go pkg/ssh/ssh.go +3 -0

No files found.
--- a/pkg/ssh/ssh.go
+++ b/pkg/ssh/ssh.go
@@ -364,6 +364,9 @@ func (l *SSHTunnelList) healthCheck(e sshTunnelEntry) error {
 		Dial: e.Tunnel.Dial,
 		// TODO(cjcullen): Plumb real TLS options through.
 		TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
+		// We don't reuse the clients, so disable the keep-alive to properly
+		// close the connection.
+		DisableKeepAlives: true,
 	})
 	client := &http.Client{Transport: transport}
 	resp, err := client.Get(l.healthCheckURL.String())