在 Polly 重试策略中重用 HttpRequestMessage

Question

一个HttpRequestMessage object 只能使用一次； 以后尝试使用相同的 object 会引发异常。 我正在使用 Polly 重试一些请求，我遇到了这个问题。 我知道如何克隆请求，有很多关于 SO 的示例，但我不知道如何克隆请求并在 Polly 重试时发送新请求。 我怎样才能做到这一点？

这些是我的政策，供参考。 这是一个 Xamarin 应用程序。 如果出现网络故障，我想重试几次，如果响应未经授权，我想使用保存的凭据重新验证并再次尝试原始请求。

public static PolicyWrap<HttpResponseMessage> RetryPolicy
{
    get => WaitAndRetryPolicy.WrapAsync(ReAuthPolicy);
}

private static IAsyncPolicy WaitAndRetryPolicy
{
    get => Policy.Handle<WebException>().WaitAndRetryAsync(4, _ => TimeSpan.FromSeconds(2));
}

private static IAsyncPolicy<HttpResponseMessage> ReAuthPolicy
{
    get => Policy.HandleResult<HttpResponseMessage>(x => x.StatusCode == HttpStatusCode.Unauthorized)
        .RetryAsync((_, __) => CoreService.LogInWithSavedCredsAsync(true));
}

由于HttpRequestMessage重用，这不起作用，但这是我想要完成的：

var request = new HttpRequestMessage(HttpMethod.Post, "some_endpoint")
{
    Content = new StringContent("some content")
};

request.Content.Headers.ContentType = new MediaTypeHeaderValue("application/json");

var policyResponse = await ConnectivityHelper.RetryPolicy
    .ExecuteAndCaptureAsync(() => _client.SendAsync(request)).ConfigureAwait(false);

// handle outcome

Answer 1

如果HttpRequestMessage被重用，则抛出InvalidOperationException的代码是HttpClient本身中的验证步骤。

    private static void CheckRequestMessage(HttpRequestMessage request)
    {
        if (!request.MarkAsSent())
        {
            throw new InvalidOperationException(SR.net_http_client_request_already_sent);
        }
    }

见https://github.com/microsoft/referencesource/blob/e7b9db69533dca155a33f96523875e9c50445f44/System/net/System/Net/Http/HttpClient.cs#L628-L634

    internal bool MarkAsSent()
    {
        return Interlocked.Exchange(ref sendStatus, messageAlreadySent) == messageNotYetSent;
    }

见https://github.com/microsoft/referencesource/blob/5697c29004a34d80acdaf5742d7e699022c64ecd/System/net/System/Net/Http/HttpRequestMessage.cs#L193

您可以将 polly 重试策略放在DelegatingHandler ，这样就可以了。 它还提供了一个很好的 SoC（关注点分离）。 如果在未来，你想不重试或改变重试的行为，你只需删除DelegatingHandler或改变它。 注意处理HttpRequestMessage和中间HttpResponseMessage对象。

Answer 2

嗯，最简单的解决方案是将HttpRequestMessage的创建移动到ExecuteAndCaptureAsync委托中。 换句话说，不要重用而是重新创建它：

var policyResponse = await ConnectivityHelper.RetryPolicy
    .ExecuteAndCaptureAsync(async () => {
    var request = new HttpRequestMessage(HttpMethod.Post, "some_endpoint")
    {
        Content = new StringContent("some content", Encoding.UT8, "application/json")
    };
    return await _client.SendAsync(request)).ConfigureAwait(false);
});

或者只是更喜欢PostAsync而不是SendAsync

var policyResponse = await ConnectivityHelper.RetryPolicy.ExecuteAndCaptureAsync(
  async () => 
    await _client.PostAsync("some_endpoint", 
         new StringContent("some content", Encoding.UT8, "application/json"))
    .ConfigureAwait(false)
});