Golang中必须关闭Response Body的原因与解决方法

Golang中必须关闭Response Body的原因与解决方法 以下代码片段触发了 linter 警告。请帮我解决这个 linter 问题。

if something {

        tr := &http.Transport{TLSClientConfig: config}

        client := &http.Client{Transport: tr}

        resp, err = client.Get(x5u) // Linter 警告：必须关闭响应体

    } else {

        resp, err = http.Get(x5u)

    }

    if err != nil {

        return nil, err

    }

defer resp.Body.Close()

更多关于Golang中必须关闭Response Body的原因与解决方法的实战教程也可以访问 https://www.itying.com/category-94-b0.html

6 回复

具体在哪个位置？最佳做法是在分配 resp 之后立即使用 defer。

更多关于Golang中必须关闭Response Body的原因与解决方法的实战系列教程也可以访问 https://www.itying.com/category-94-b0.html


这次忽略 linter 的警告是可以的。defer 语句的位置完全正确。

即使我在错误检查之前放置了 defer，仍然收到了相同的警告。

你的代码路径中存在不会关闭 resp.Body() 的情况。如果由于某些原因 err 不为 nil，那么你的 defer 将永远不会被执行，因此响应体会保持打开状态。

如果发生错误，你不需要调用 defer resp.Body.Close()。参考

在Go中必须关闭HTTP响应体的主要原因是为了释放网络连接资源，避免内存泄漏。即使不读取响应体，也必须显式关闭它。

您的代码存在一个潜在问题：当something为false时，如果http.Get()调用失败，resp会是nil，此时调用defer resp.Body.Close()会导致panic。

以下是修复后的代码：

var resp *http.Response
var err error

if something {
    tr := &http.Transport{TLSClientConfig: config}
    client := &http.Client{Transport: tr}
    resp, err = client.Get(x5u)
} else {
    resp, err = http.Get(x5u)
}

if err != nil {
    return nil, err
}

// 确保resp不为nil后再关闭Body
defer resp.Body.Close()

// 继续处理响应...

更健壮的写法是使用defer配合条件判断：

var resp *http.Response
var err error

if something {
    tr := &http.Transport{TLSClientConfig: config}
    client := &http.Client{Transport: tr}
    resp, err = client.Get(x5u)
} else {
    resp, err = http.Get(x5u)
}

if err != nil {
    return nil, err
}

defer func() {
    if resp != nil {
        resp.Body.Close()
    }
}()

// 读取响应体
body, err := io.ReadAll(resp.Body)
if err != nil {
    return nil, err
}

// 处理响应数据...

如果只需要确保资源释放而不关心响应内容，可以这样处理：

resp, err := http.Get("https://example.com")
if err != nil {
    return err
}
defer resp.Body.Close()

// 只读取少量字节或直接丢弃
_, _ = io.Copy(io.Discard, resp.Body)

关键点：

  1. HTTP响应体必须关闭以释放底层连接
  2. 在错误处理路径中，如果响应为nil则不能调用Close()
  3. 即使不读取响应体，也需要消耗或关闭它来重用连接
回到顶部