我们正在为 HttpClient 创建一个包装器。因为我们将遵循 https://github.com/mspnp/performance-optimization 中的性能优化指导.我们希望避免反模式 - 该文档中提到的不正确实例化。我将此指南推荐给我的团队以使用静态 HttpClient。我得到的反馈是关于线程安全的。每个请求都有一个包含用户声明的 header 。由于我有一个静态 HttpClient,它会是线程安全的吗?如果我们有多个请求同时访问代码(例如 GET),设置 header 会是竞争条件吗?我们有如下实现。
public class HttpClientHelper{
private static readonly HttpClient _HttpClient;
static HttpClientHelper() {
HttpClient = new HttpClient();
HttpClient.Timeout = TimeSpan.FromMinutes(SOME_CONFIG_VALUE);
}
public async Task<HttpResponseMessage> CallHttpClientPostAsync(string requestUri, HttpContent requestBody)
{
AddHttpRequestHeader(httpClient);
var response = await httpClient.PostAsync(requestUri, requestBody); //Potential thread synchronization issue???
return response;
}
public HttpResponseMessage CallHttpClientGet(string requestUri)
{
AddHttpRequestHeader(httpClient);
var response = httpClient.GetAsync(requestUri).Result; //Potential thread synchronization issue???
return response;
}
private void AddHttpRequestHeader(HttpClient client)
{
string HeaderName = "CorrelationId";
client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue(Properties.Settings.Default.HttpClientAuthHeaderScheme, GetTokenFromClaims()); //Race condition???
if (client.DefaultRequestHeaders.Contains(HeaderName))
client.DefaultRequestHeaders.Remove(HeaderName);
client.DefaultRequestHeaders.Add(HeaderName, Trace.CorrelationManager.ActivityId.ToString());
}
}
最佳答案
您的团队是正确的,这远非线程安全。考虑这种情况:
更好的方法是让您的 CallXXX 方法创建新的 HttpRequestMessage对象,并在这些对象上设置标题,然后使用 HttpClient.SendAsync调用电话。
还要记住,重复使用
HttpClient
实例仅在您对同一主机进行多次调用时才有用。
关于asp.net - ASP.net HttpRequest 上的静态 HttpClient 线程安全,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/29762390/