From 2d21248a780495ce36c40256e226428dbf3598e2 Mon Sep 17 00:00:00 2001 From: Sleeyax Date: Wed, 26 Feb 2025 23:45:15 +0100 Subject: [PATCH] Remove `content-length` header if present The content length is already handled internally. Leaving it specified causes strange `400 bad request` errors from the destination server. See: https://github.com/bogdanfinn/tls-client/issues/147. --- src-go/server/server.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src-go/server/server.go b/src-go/server/server.go index 5bcc6cc..5cda47f 100644 --- a/src-go/server/server.go +++ b/src-go/server/server.go @@ -77,6 +77,9 @@ func StartServer(addr string) error { req.URL.Scheme = config.Scheme req.RequestURI = "" req.Header[fhttp.HeaderOrderKey] = config.HeaderOrder + // The content-length header is already set by the client (internally). + // Leaving it here causes strange '400 bad request' errors from the destination, so we remove it. + req.Header.Del("Content-Length") res, err := client.Do(req) if err != nil {