4

我们使用HttpClient并行向远程 Web API 发送请求：

public async Task<HttpResponseMessage> PostAsync(HttpRequestInfo httpRequestInfo)
{
    using (var httpClient = new HttpClient())
    {
        httpClient.BaseAddress = new Uri(httpRequestInfo.BaseUrl);
        if (httpRequestInfo.RequestHeaders.Any())
        {
            foreach (var requestHeader in httpRequestInfo.RequestHeaders)
            {
                httpClient.DefaultRequestHeaders.Add(requestHeader.Key, requestHeader.Value);
            }
        }

        return await httpClient.PostAsync(httpRequestInfo.RequestUrl, httpRequestInfo.RequestBody);
    }
}

该 API 可以被多个线程同时调用。运行大约四个小时后，我们发现发生了内存泄漏问题，从 profiling 工具来看，似乎有两个ServicePoint对象，其中一个相当大，大约 160 MB。

据我所知，我可以看到代码上面的一些问题：

  • HttpClient我们应该尽可能地共享实例。在我们的例子中，请求地址和标头可能会有很大差异，那么这是我们可以做点什么还是不会损害太多性能？我只是想到我们可以准备一个字典来存储和查找HttpClient实例。
  • 我们没有修改DefaultConnectionLimitof ServicePoint，所以默认只能同时向同一个服务器发送两个请求。如果我们把这个值改大一点，内存泄漏问题就可以解决了吗？
  • 我们还抑制了 HTTPS 证书验证：ServicePointManager.ServerCertificateValidationCallback = delegate { return true; };这与问题有关吗？

由于这个问题不容易重现（需要很多时间），我只需要一些想法，以便我可以优化我们的代码以长时间运行。

4

1 回答 1

2

自己解释一下情况，以防其他人以后也遇到这个问题。

首先，这不是内存泄漏，而是性能问题。

我们在打开代理的虚拟机中测试我们的应用程序。它导致互联网很慢。所以在我们的例子中，每个 HTTP 请求可能会花费 3-4 秒。ServicePoint随着时间的推移，队列中会有很多连接。因此，这不是内存泄漏，那只是因为之前的连接没有足够快地完成。

只要确保每个 HTTP 请求都没有那么慢，一切都会变得正常。

我们还尝试减少HttpClient实例，以提高 HTTP 请求性能：

private readonly ConcurrentDictionary<HttpRequestInfo, HttpClient> _httpClients;

private HttpClient GetHttpClient(HttpRequestInfo httpRequestInfo)
{
    if (_httpClients.ContainsKey(httpRequestInfo))
    {
        HttpClient value;
        if (!_httpClients.TryGetValue(httpRequestInfo, out value))
        {
            throw new InvalidOperationException("It seems there is no related http client in the dictionary.");
        }

        return value;
    }

    var httpClient = new HttpClient { BaseAddress = new Uri(httpRequestInfo.BaseUrl) };
    if (httpRequestInfo.RequestHeaders.Any())
    {
        foreach (var requestHeader in httpRequestInfo.RequestHeaders)
        {
            httpClient.DefaultRequestHeaders.Add(requestHeader.Key, requestHeader.Value);
        }
    }

    httpClient.DefaultRequestHeaders.ExpectContinue = false;
    httpClient.DefaultRequestHeaders.ConnectionClose = true;
    httpClient.Timeout = TimeSpan.FromMinutes(2);

    if (!_httpClients.TryAdd(httpRequestInfo, httpClient))
    {
        throw new InvalidOperationException("Adding new http client thrown an exception.");
    }

    return httpClient;
}

在我们的例子中，对于同一个服务器地址，只有请求正文不同。我还覆盖Equals了GetHashCode.HttpRequestInfo

同时，我们设置ServicePointManager.DefaultConnectionLimit = int.MaxValue;

希望这可以帮助你。

于 2015-03-19T02:04:52.780 回答