Adding client-level cookie container

[alexeyzimarev]

• Added a custom cookie container to RestClientOptions
• It will not be added to the message handler
• Cookies will be added as headers the same way as request cookies

[kendallb]

My concern would be adding the cookies to the container in the request because if the user is using a request specific container that they are maintaining, it means the client ones will end up in that container after the request.

cookieContainer.Add(Options.CookieContainer.GetCookies(url));

Is it feasible to simply call headers.AddCookieHeaders(cookieContainer, url) for both cookie containers rather than combine them?

[alexeyzimarev]

Yeah, I thought about that as one of the options, but I haven't considered the request container concern. You are right, will change it now.

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	c121623
Status:	✅  Deploy successful!
Preview URL:	https://92532ab6.restsharp.pages.dev
Branch Preview URL:	https://client-level--cookies.restsharp.pages.dev

View logs

[alexeyzimarev]

@kendallb I updated it following your comment.

[kendallb]

Looks good to me