From 87e7722a76959339df9af0660bacc1a9fe7347be Mon Sep 17 00:00:00 2001 From: Dominic Evans Date: Fri, 11 Aug 2023 22:40:51 +0100 Subject: [PATCH] fix(fvt): disable keepalive on toxiproxy client We don't want to keep the tcp conn alive as we only make very few requests and it'll show up as a leaked goroutine in-between tests. Signed-off-by: Dominic Evans --- internal/toxiproxy/client.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/internal/toxiproxy/client.go b/internal/toxiproxy/client.go index e916bcbcac..a30557048b 100644 --- a/internal/toxiproxy/client.go +++ b/internal/toxiproxy/client.go @@ -24,7 +24,8 @@ func NewClient(endpoint string) *Client { KeepAlive: 30 * time.Second, }).DialContext, ForceAttemptHTTP2: true, - MaxIdleConns: 100, + MaxIdleConns: -1, + DisableKeepAlives: true, IdleConnTimeout: 90 * time.Second, TLSHandshakeTimeout: 10 * time.Second, ExpectContinueTimeout: 1 * time.Second,