What could happen if I don't close response.Body?
What happens in this case? will there be a memory leak?
It's a resource leak. The connection won't be re-used, and can remain open in which case the file descriptor won't be freed.
Also is it safe to put in defer resp.Body.Close() immediately after getting the response object?
No, follow the example provided in the documentation and close it immediately after checking the error.
client := http.DefaultClient
resp, err := client.Do(req)
if err != nil {
return nil, err
}
defer resp.Body.Close()
From the http.Client
documentation:
If the returned error is nil, the Response will contain a non-nil Body which the user is expected to close. If the Body is not both read to EOF and closed, the Client's underlying RoundTripper (typically Transport) may not be able to re-use a persistent TCP connection to the server for a subsequent "keep-alive" request.
If Response.Body
won't be closed with Close()
method than a resources associated with a fd won't be freed. This is a resource leak.
Closing Response.Body
From response source:
It is the caller's responsibility to close Body.
So there is no finalizers bound to the object and it must be closed explicitly.
Error handling and deferred cleanups
On error, any Response can be ignored. A non-nil Response with a non-nil error only occurs when CheckRedirect fails, and even then the returned Response.Body is already closed.
resp, err := http.Get("http://example.com/")
if err != nil {
// Handle error if error is non-nil
}
defer resp.Body.Close() // Close body only if response non-nil
At first the descriptor never closes, as things mentioned above.
And what's more, golang will cache the connection (using persistConn
struct to wrap) for reusing it, if DisableKeepAlives
is false.
In golang after use client.Do
method, go will run goroutine readLoop
method as one of the step.
So in golang http transport.go
, a pconn(persistConn struct)
won't be put into idleConn
channel until the req canceled in the readLoop
method, and also this goroutine(readLoop
method) will be blocked until the req canceled.
Here is the code showing it.
If you want to know more, you need to see the readLoop
method.