dotnet/aspnetcore

TestServer.CreateClient : the HttpClient created didn't support well the cancellation.

Open

#5,938 opened on Sep 17, 2018

View on GitHub
 (4 comments) (1 reaction) (0 assignees)C# (37,933 stars) (10,653 forks)batch import
affected-fewarea-hostingarea-networkingbughelp wantedseverity-minor

Description

Hello,

I just found a nasty bug on HttpClient created from TestServer. When I call a request which took 2 seconds if I cancel it after one second. The client created by test server didn't throw an OperationCancelledException.

Here the code with real httpClient :

var httpClient = new HttpClient();
using (var cts = new CancellationTokenSource(TimeSpan.FromSeconds(1)))
{
    var request = new HttpRequestMessage
    {
        Method = HttpMethod.Get,
        RequestUri = new Uri("http://localhost:4242/TimeoutTest/Action2Secs")
     };
     await httpClient.SendAsync(request, HttpCompletionOption.ResponseContentRead, cts.Token);
}

The same code with client created by the class TestServer.

var httpClient = new testServer.CreateClient();
using (var cts = new CancellationTokenSource(TimeSpan.FromSeconds(1)))
{
    var request = new HttpRequestMessage
    {
        Method = HttpMethod.Get,
        RequestUri = new Uri("http://localhost:4242/TimeoutTest/Action2Secs")
     };
     await httpClient.SendAsync(request, HttpCompletionOption.ResponseContentRead, cts.Token);
}

Contributor guide