API Threading issue - Concurrent connection cancelling the API Thread

Question

API Threading issue - Concurrent connection cancelling the API Thread

Sadachar, Abhijeet (Cognizant) 1

Hi Team,

We are using ASP.NET Core 3.1 and making API calls using httpclient with "GetASync" and "PostAsync" methods. During concurrent calls, the API thread is getting terminated / cancelled with below exception. Could you please help to know how to handle this error and resolve this issue for concurrent user calls?

Code Sample:

var clientt = new HttpClient();

client.Timeout = Timespan.FromSeconds (120);

clientt.DefaultRequestHeaders.Accept.Add(new MediaTypeWithQualityHeaderValue("application/json"));

var response = clientt.PostAsync(uri, content).Result;

response.Wait();

string resultString = response.Content.ReadAsStringASync().result;

Exception:
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions) at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken) at System.Threading.Tasks.Task.Wait()

AgaveJoe 31,361 Reputation points

2021-10-14T11:44:52.48+00:00

There are a lot of problems with your code. HttpClient should be declared once at the application level for each service. The reference documentation illustrates how to use HttpClient properly.

HttpClient Class
Use IHttpClientFactory to implement resilient HTTP requests

Asynchronous method should use await not Result. Your design turns asynchronous execution into synchronous which blocks the main thread and locks your application. It seems like you did not read the docs...

Asynchronous programming with async and await

1 answer

Your answer

AgaveJoe 31,361 Reputation points

2021-10-14T11:44:52.48+00:00

There are a lot of problems with your code. HttpClient should be declared once at the application level for each service. The reference documentation illustrates how to use HttpClient properly.

HttpClient Class
Use IHttpClientFactory to implement resilient HTTP requests

Asynchronous method should use await not Result. Your design turns asynchronous execution into synchronous which blocks the main thread and locks your application. It seems like you did not read the docs...

Asynchronous programming with async and await

Answer 1

Bruce (SqlWork.com) 84,071

also the error indicates the 120 second timeout fired and canceled the task. maybe the site you are calling is as poorly written as your sample code.

0 comments

Share via

API Threading issue - Concurrent connection cancelling the API Thread

1 answer

Your answer