zap-api-dotnet icon indicating copy to clipboard operation
zap-api-dotnet copied to clipboard

Enable cookies on WebClient.

Open binarymist opened this issue 8 years ago • 2 comments

If ClientApi.PostToUrl is not generic enough for main stream consumption, consumers could make an extension method out of it. Access to the WebClient's headers are required in ClientApi.PostToUrl Currently consumers have to maintain this locally. Once this is included, consumers can then depend on the NuGet package instead.

binarymist avatar Jun 26 '16 04:06 binarymist

Hi, Thanks for the pull request. It took me a little while to get around this pull request.

  1. Do you have a sample of how the public method ClientApi.PostToUrl can come in handy for the api consumers?
  2. What is the use case for this method?

It seems like in order to support this method ClientApi.PostToUrl (and also send some cookie information along the request) we need to restructure the IWebClient interface rather than directly modifying the SystemWebClient class.

gmaran23 avatar Jul 21 '16 18:07 gmaran23

I no longer have access to this .NET environment, and because it's been so long, is no longer fresh in my head. I've passed this onto my client that I was working for at the time. My client will hopefully provide feedback soon.

binarymist avatar Jul 26 '16 14:07 binarymist