Cache default httpclient

[nillis]

No description provided.

[thinkingserious]

Hello @nillis,

Thank you for the PR!

We will be reviewing this PR soon, in the mean time, if you have not, please be sure to sign our CLA so that we can merge your changes. Thanks again!

Team DX

[nillis]

#17

[thinkingserious]

Hi @nillis,

I believe the test is failing because private static HttpClient _httpClient = new HttpClient(); should be in the Client class rather than the Response class.

[nillis]

Idd, my bad. New build is running :)

[thinkingserious]

Perfect, thanks for the quick fix!

Now, we just need a signed CLA from you in order to review and merge. Thanks again!

[pkarlapu]

Hey..Now that this is merged to master, I am wondering when would a new version of NuGet be released.

[thinkingserious]

Hi @karlapudipradeep,

You should be able to access this version here: https://www.nuget.org/packages/SendGrid.CSharp.HTTP.Client/3.1.0

With Best Regards,

Elmer

[pkarlapu]

@thinkingserious Cool..Thanks! I mistook this commit as a commit to https://github.com/sendgrid/sendgrid-csharp repo and was excited for a little bit. It seems like there is an issue opened for the other repo at sendgrid/sendgrid-csharp#405. I will try sending a pull request for it when I get a chance.

[thinkingserious]

@pkarlapu,

I'm actually reviewing that PR right now: sendgrid/sendgrid-csharp#417 :)

Your feedback would be greatly appreciated. Thanks!