Statische HttpClient Thread sicher auf ASP.net HttpRequest

Question

Wir erstellen einen Wrapper für HttpClient. Wir werden die Leitlinien zur Leistungsoptimierung von https://github.com/mspnp/performance-optimization befolgen. Wir wollen Anti-Pattern vermeiden - Unsaubere Instanziierung, die in diesem Dokument erwähnt wird. Ich habe diese Anleitung an mein Team weitergeleitet, um den statischen HttpClient zu verwenden. Das Feedback, das ich bekommen habe, ist auf Thread-Sicherheit. Jede Anfrage hat einen Header, der den Benutzeranspruch enthält. Da ich einen statischen HttpClient habe, wird es Thread-sicher sein? Wenn wir mehrere Anfragen gleichzeitig auf den Code (zum Beispiel GET) haben, wird es eine Race Condition sein, Header zu setzen? Wir haben die Implementierung wie folgt.

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()); 
} 

}

Answer 1

Ihr Team ist richtig, das ist noch lange Thread-sicher. Stellen Sie sich dieses Szenario vor:

• Thread A setzt CorrelationId-Header auf "foo".
• Thread B setzt CorrelationId-Header auf "bar".
• Thread A sendet Anforderung, die CorrelationId von Thread B enthält.

Ein besserer Ansatz wäre für Ihre CallXXX Methoden neue HttpRequestMessage Objekte zu erstellen, und stellen Sie den Header auf diejenigen, und verwenden Sie HttpClient.SendAsync den Anruf zu tätigen.

Beachten Sie auch, dass die erneute Verwendung von HttpClient Instanzen nur dann von Vorteil ist, wenn Sie mehrere Aufrufe an denselben Host vornehmen.