java - OkHttp : avoid leaked connection warning

我正在使用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();
}

我的应用程序大量使用网络，但该警告频繁出现。我从未观察到任何由此推测的泄漏引起的问题，但是我仍然想了解我是否做错了什么以及做错了什么。

有人能对此有所启示吗？

最佳答案

通过升级到OkHttp 3.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();
}