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
Add SKIP_HEADER for skipping automatically added headers #2018
Conversation
Co-authored-by: hodbn <hodbn@users.noreply.github.com>
Thanks @hodbn for the test case, I added you as a coauthor to the patch you sent me :) |
6bc1f22
to
5ac4e44
Compare
5ac4e44
to
a66c224
Compare
Codecov Report
@@ Coverage Diff @@
## master #2018 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 25 25
Lines 2290 2298 +8
=========================================
+ Hits 2290 2298 +8
Continue to review full report at Codecov.
|
@hodbn Thanks for the review, good catch on the copy issue. |
Closes #2014, Closes #2021
Changed the sentinel from
SUPPRESS_USER_AGENT
toSKIP_HEADER
for skipping 3 of the automatically added headersAccept-Encoding
,Host
, andUser-Agent
.cc @nateprewitt could you test this to ensure it works as you're expecting?