Co może się stać, jeśli nie zamknę odpowiedzi.

W Go mam kilka odpowiedzi http i czasami zapominam zadzwonić:

resp.Body.Close()

Co się dzieje w tym przypadku? czy nastąpi wyciek pamięci? Czy jest też bezpieczne wprowadzenie defer resp.Body.Close()natychmiast po otrzymaniu obiektu odpowiedzi?

client := http.DefaultClient
resp, err := client.Do(req)
defer resp.Body.Close()
if err != nil {
    return nil, err
}

Co się stanie, jeśli wystąpi błąd, respczy może resp.Bodybyć zerowy?

Co się dzieje w tym przypadku? czy nastąpi wyciek pamięci?

To wyciek zasobów. Połączenie nie zostanie ponownie użyte i może pozostać otwarte, w takim przypadku deskryptor pliku nie zostanie zwolniony.

Czy jest również bezpieczne wprowadzenie defer lub Body.Close () natychmiast po uzyskaniu obiektu odpowiedzi?

Nie, postępuj zgodnie z przykładem podanym w dokumentacji i zamknij go natychmiast po sprawdzeniu błędu.

client := http.DefaultClient
resp, err := client.Do(req)
if err != nil {
    return nil, err
}
defer resp.Body.Close()

Z http.Clientdokumentacji:

Jeśli zwrócony błąd wynosi zero, odpowiedź będzie zawierać niezerową treść, którą użytkownik powinien zamknąć. Jeśli treść nie jest jednocześnie odczytywana do EOF i zamykana, bazowy RoundTripper klienta (zazwyczaj Transport) może nie być w stanie ponownie użyć trwałego połączenia TCP z serwerem dla kolejnego żądania „utrzymywania aktywności”.

Jeśli Response.Bodynie zostanie zamknięte Close()metodą, wówczas zasoby powiązane z fd nie zostaną zwolnione. To wyciek zasobów.

Zamknięcie Response.Body

Ze źródła odpowiedzi :

Obowiązkiem dzwoniącego jest zamknięcie Ciała.

Nie ma więc finalizatorów związanych z obiektem i musi być jawnie zamknięty.

Obsługa błędów i odroczone czyszczenie

W przypadku błędu każdą odpowiedź można zignorować. Niezerowa odpowiedź z błędem różnym od zera występuje tylko wtedy, gdy CheckRedirect nie powiedzie się, a nawet wtedy zwrócona odpowiedź Response.Body jest już zamknięta.

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

Początkowo deskryptor nigdy się nie zamyka, jak wspomniano powyżej.

Co więcej, golang buforuje połączenie (używając persistConnstruct to wrap) do ponownego użycia, jeśli DisableKeepAlivesjest fałszywe.

W golang po użyciu client.Dometody, go uruchomi readLoopmetodę goroutine jako jeden z kroków.

Więc w golang http transport.go, a pconn(persistConn struct)nie zostanie wstawiony do idleConnkanału, dopóki req nie zostanie anulowany w readLoopmetodzie, a także ta goroutine ( readLoopmetoda) zostanie zablokowana do czasu anulowania żądania .

Oto kod, który to pokazuje.

Jeśli chcesz wiedzieć więcej, musisz zapoznać się z readLoopmetodą.

Zobacz https://golang.org/src/net/http/client.go
„Gdy błąd jest równy zero, resp zawsze zawiera wartość inną niż zero lub Body.”

ale nie mówią, kiedy err! = nil, resp zawsze jest nil.
Dalej mówią: „Jeśli odpowiednie ciało nie jest zamknięte, bazowy RoundTripper klienta (zazwyczaj Transport) może nie być w stanie ponownie użyć trwałego połączenia TCP z serwerem dla kolejnego żądania„ utrzymywania aktywności ”.

Więc zazwyczaj rozwiązałem problem w następujący sposób:

client := http.DefaultClient
resp, err := client.Do(req)
if resp != nil {
   defer resp.Body.Close()
}
if err != nil {
    return nil, err 
}