我正在使用OKHTTP 3,并且不断收到泄漏的连接警告:

WARNING: A connection to https://help.helpling.com/ was leaked. Did you forget to close a response body?
Jul 14, 2016 6:57:09 PM okhttp3.ConnectionPool pruneAndGetAllocationCount

每次我得到一个ResponseBody时,我要么调用.string()来为我关闭流,要么在一个finally块中显式关闭流,方法如下:

ResponseBody responseBody = response.body();
try (Reader responseReader = responseBody.charStream()) {
    ...
}
finally {
    responseBody.close();
}

我的应用程序对网络进行了大量的使用,但是这种警告经常出现。我从来没有注意到由这个假定的泄漏引起的任何问题,但我仍然想知道我是否做错了什么以及做错了什么。
有人能解释一下吗?

最佳答案:

通过升级到okhtp3.7,Eclipse开始警告我潜在的资源泄漏。我发现我的问题在于我写的这个方法:

public static Response getResponse(HttpUrl url, OkHttpClient client) throws IOException {
    Builder request = new Request.Builder().url(url);
    Response response = client.newCall(request.build()).execute();
    if (!response.isSuccessful()) {
        boolean repeatRequest = handleHttpError(response);
        if (repeatRequest)
            return getResponse(url, client, etag);
        else
            throw new IOException(String.format("Cannot get successful response for url %s", url));
    }
    return response;
}

我假设总是调用getResponse(url, client).body().string()流将自动关闭。但是,每当响应失败时,在执行.string()之前会引发异常,因此流将保持打开状态。
在响应失败的情况下添加显式关闭解决了问题。

if (!response.isSuccessful()) {
    boolean repeatRequest = handleHttpError(response);
    response.close();
}
Logo

腾讯云面向开发者汇聚海量精品云计算使用和开发经验,营造开放的云计算技术生态圈。

更多推荐