Always drain body of http.Request might not be required

[charconstpointer]

I think this might not be the best advice, since it seems like Close() on the *body discards it for you by default

https://github.com/golang/go/blob/9d23975d89e6cc3df4f2156b2ae0df5d2cef16fb/src/net/http/transfer.go#L979

• always discard body e.g. io.Copy(ioutil.Discard, resp.Body) if you don't use it
  • HTTP client's Transport will not reuse connections unless the body is read to completion and closed

Am I missing something?

[cristaloleg]

Yeah, probably it's oudated. I will take a look this weekend.