使用HttpClient同步调用异步方法如何避免死锁，修改后返回400错误怎么办

问题原因 #

修改后触发400 Bad Request的常见原因如下：

• uri变量作用域问题：修改后的方法是private static静态方法，如果uri是类的非静态成员，静态方法访问到的会是未正确赋值的默认值，导致请求地址错误
• 请求上下文丢失：如果运行在ASP.NET (.NET Framework)环境下，Task.Run会脱离原始请求上下文，如果接口依赖上下文自动携带的Cookie、身份认证头等信息，新线程中这些信息不会自动附加到请求中，服务端校验失败就会返回400
• 两个版本的代码都存在重复创建HttpClient实例的问题，会导致socket资源耗尽，线上运行会出现偶发请求失败

无死锁风险的同步调用实现 #

不需要用Task.Run绕实现，通过ConfigureAwait(false)切断上下文捕获即可避免死锁，正确写法如下：

// HttpClient定义为静态全局实例，避免重复创建
private static readonly HttpClient _httpClient = new HttpClient();

// 静态构造函数，全局只需执行一次配置
static 你的类名()
{
    System.Net.ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12 | SecurityProtocolType.Tls11;
}

public HttpResponseMessage SendData(string jsonData)       
{
    // ConfigureAwait(false) 禁止捕获同步上下文，避免死锁
    HttpResponseMessage resp = _httpClient.PostAsync(uri, new StringContent(jsonData, Encoding.UTF8, "application/json"))
        .ConfigureAwait(false)
        .GetAwaiter()
        .GetResult();

    if (resp.IsSuccessStatusCode)
    {
        // 处理成功逻辑
    }
    else
    {
        // 处理错误逻辑
    }
    return resp;
}

保留Task.Run写法的修复方案 #

如果要保留Task.Run的写法，需要手动解决地址和上下文问题：

private static readonly HttpClient _httpClient = new HttpClient();

static 你的类名()
{
    System.Net.ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12 | SecurityProtocolType.Tls11;
}

// 将uri作为参数传入，避免作用域问题
private static HttpResponseMessage SendData(string jsonData, Uri requestUri)
{
    // 如果需要携带原始请求的Cookie、认证头，在此处手动附加到HttpClient的DefaultRequestHeaders中
    Task<HttpResponseMessage> callTask = Task.Run(() => 
        _httpClient.PostAsync(requestUri, new StringContent(jsonData, Encoding.UTF8, "application/json")));
    callTask.Wait();
    return callTask.Result;
}

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