Migrate httpweb request to http client - #79
Conversation
|
Anjali Gupta (@anjali24gupta) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
4973548 to
76f6e25
Compare
| <Project Sdk="Microsoft.NET.Sdk"> | ||
| <PropertyGroup> | ||
| <TargetFrameworks>netstandard2.0;</TargetFrameworks> | ||
| <TargetFrameworks>net6.0</TargetFrameworks> |
There was a problem hiding this comment.
Can we keep netstandard2.0? that allows clients on netframework to work
Also, instead of net6, we can jump to latest LTS version
| var handler = new HttpClientHandler | ||
| { | ||
| AutomaticDecompression = DecompressionMethods.GZip | DecompressionMethods.Deflate, | ||
| AllowAutoRedirect = true, |
There was a problem hiding this comment.
What is the use case for cookie container? I don't think our service uses this
| if (customHeaders != null) | ||
| { | ||
| string contentType; | ||
| if (customHeaders.TryGetValue("Content-Type", out contentType)) |
There was a problem hiding this comment.
We can just remove this? or is the out field used?
No description provided.