search

MelbourneDeveloper / RestClient.Net

.NET REST Client Framework for all platforms
MIT License
366 stars 43 forks source link

Update Client.cs #121

Closed andreapalloni-dedalus closed 1 year ago

andreapalloni-dedalus commented 1 year ago

In constructor if serializationAdapter is null DefaultRequestHeaders are always overwritten, also if not null

MelbourneDeveloper commented 1 year ago

@andreapalloni-dedalus thankyou for this. I didn't see this PR. I got swamped with useless dependabot alerts. I'll take a look at this soon.

MelbourneDeveloper commented 1 year ago

@andreapalloni-dedalus Hi. I'm trying to dust this library off, and I noticed you closed this. Do you want to reopen this? I'm going to try to merge it.

MelbourneDeveloper commented 1 year ago

@andreapalloni-dedalus the commit in the original PR broke some tests, so I fixed it here: ead38f28fbaaaf316f034699d57355c4a2954909

But, your commit did make into the final build for version 6.0 here: 626d4460923d6843025f4202840919e84e3b71f0