Azure函数中单例HttpClient多用户不同OAuthToken的请求头线程安全疑问

你目前的两种实现都存在严重的线程安全隐患——哪怕在第一个请求后重新设置Header也没用，根源在于：

单例HttpClient的DefaultRequestHeaders是全局共享属性，所有使用该实例的请求都会复用这些头信息。当多个用户（线程）同时调用test方法时，会出现竞态条件：

• 线程A设置完自己的OAuthToken，发起第一个请求后进入Thread.Sleep(5000)
• 此时线程B过来，直接修改了全局的Authorization头为自己的token
• 线程A醒来发起第二个请求时，会误用线程B的token，导致身份验证完全混乱

你在第一个请求后重新设置Header的操作，既没法覆盖中间其他线程的修改，也完全是冗余的——只要其他线程在Sleep期间动过全局Header，你的第二个请求依然会用错凭证。

不要修改HttpClient的全局DefaultRequestHeaders，而是为每个请求单独构建HttpRequestMessage，在请求对象上设置专属Header。这样每个请求的Header都是独立隔离的，完全不会互相干扰。

修改后的代码示例（同时优化了异步调用实践）：

public class MyClass : IMyClass 
{ 
    private readonly HttpClient _httpClient; 

    public MyClass(HttpClient httpClient) 
    { 
        _httpClient = httpClient; 
    } 

    // 改用async Task避免阻塞线程，符合异步编程最佳实践
    public async Task TestAsync(string OAuthToken) 
    { 
        // 第一个API请求：单独构建请求并设置专属Header
        string firstApi = "https://GetSometthingFirst.com/processes?api-version=5.0";
        var firstRequest = new HttpRequestMessage(HttpMethod.Get, firstApi);
        firstRequest.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue("application/json"));
        firstRequest.Headers.Authorization = new AuthenticationHeaderValue("Bearer", OAuthToken);
        
        var getFirstResponse = await _httpClient.SendAsync(firstRequest);
        // 建议在这里检查响应是否成功，避免无效内容传递到下一个请求
        getFirstResponse.EnsureSuccessStatusCode();

        // 用非阻塞的Task.Delay替代Thread.Sleep，不占用线程资源
        await Task.Delay(5000); 

        // 第二个API请求：同样单独构建请求
        string secondApi = "https://GetSometthingSecond.com/processes?api-version=5.0";
        var secondRequest = new HttpRequestMessage(HttpMethod.Post, secondApi);
        secondRequest.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue("application/json"));
        secondRequest.Headers.Authorization = new AuthenticationHeaderValue("Bearer", OAuthToken);
        
        // 注意：原代码中GetFirst.ToString()只会返回响应对象的类型名，需要读取实际响应内容
        var responseContent = await getFirstResponse.Content.ReadAsStringAsync();
        var content = new StringContent(responseContent, Encoding.UTF8, "application/json");
        secondRequest.Content = content;
        
        var resultResponse = await _httpClient.SendAsync(secondRequest);
        resultResponse.EnsureSuccessStatusCode();
    } 
}

1. 弃用.Result阻塞调用：async/await模式不会浪费线程资源，也能让代码逻辑更清晰，避免死锁风险。
2. 永远不要全局设置请求头：单例HttpClient的优势是复用连接池，但全局Header会成为多线程场景的定时炸弹，所有请求级别的自定义Header都应该绑定到HttpRequestMessage上。
3. 正确处理响应内容：原代码中的GetFirst.ToString()无法获取实际的API返回值，必须通过Content.ReadAsStringAsync()读取响应体。

内容的提问来源于stack exchange，提问作者user2530833