Migrate from Newtonsoft.Json to System.Text.Json
Everything is now truly async, no more fake Task.Factory.StartNew async.
Also, in commit 6a87013e18ed0f02c9e296a53f1707e538e4780c I improved all the HTTP requests to deserialize the JSON payload directly from the HTTP response stream instead of reading the response as string then deserializing to model objects. This is an improvement in both speed and memory allocations.
Also, in commit 6a87013 I improved all the HTTP requests to deserialize the JSON payload directly from the HTTP response stream instead of reading the response as string then deserializing to model objects. This is an improvement in both speed and memory allocations.
Why not just use HttpClientJsonExtensions with source generators?
I'm missing what the added value is for having generic methods with no constraints for making the HTTP requests.
Why not just use HttpClientJsonExtensions with source generators?
I don't remember but it was probably because of the dozen of MakeRequestAsync variations that depend on ApiResponseErrorHandlingDelegate which whould have been a way too risky refactoring and also nearly impossible to review.