From a2be1aabfa4b93b00a09b1b39e87ad51afb80cd5 Mon Sep 17 00:00:00 2001 From: Mihai Parparita Date: Wed, 8 Mar 2023 15:24:22 -0800 Subject: [PATCH] logtail: remove unncessary response read Effectively reverts #249, since the server side was fixed (with #251?) to send a 200 OK/content-length 0 response. Signed-off-by: Mihai Parparita --- logtail/logtail.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/logtail/logtail.go b/logtail/logtail.go index 1e2755d46..cd01d5126 100644 --- a/logtail/logtail.go +++ b/logtail/logtail.go @@ -463,14 +463,6 @@ func (l *Logger) upload(ctx context.Context, body []byte, origlen int) (uploaded return uploaded, fmt.Errorf("log upload of %d bytes %s failed %d: %q", len(body), compressedNote, resp.StatusCode, b) } - // Try to read to EOF, in case server's response is - // chunked. We want to reuse the TCP connection if it's - // HTTP/1. On success, we expect 0 bytes. - // TODO(bradfitz): can remove a few days after 2020-04-04 once - // server is fixed. - if resp.ContentLength == -1 { - resp.Body.Read(make([]byte, 1)) - } return true, nil }