-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[confighttp] Add support for cookies #10176
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10176 +/- ##
=======================================
Coverage 92.46% 92.46%
=======================================
Files 387 387
Lines 18246 18252 +6
=======================================
+ Hits 16871 16877 +6
Misses 1028 1028
Partials 347 347 ☔ View full report in Codecov by Sentry. |
if hcs.CookiesEnabled { | ||
jar, err = cookiejar.New(nil) | ||
if err != nil { | ||
return nil, err |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line cannot be tested, the New method returns nil always for now. I am just checking the error returned in case this changes in the future.
@mx-psi are we good to go? |
Description
Add support for cookies in HTTP clients with
cookies::enabled
.Link to tracking issue
Fixes #10175
Testing
Unit test
Documentation
Added to README