Skip to content

Don't send 'User-Agent' twice if header is binary#2047

Merged
sethmlarson merged 1 commit intourllib3:masterfrom
sethmlarson:user-agent-sent-twice
Nov 11, 2020
Merged

Don't send 'User-Agent' twice if header is binary#2047
sethmlarson merged 1 commit intourllib3:masterfrom
sethmlarson:user-agent-sent-twice

Conversation

@sethmlarson
Copy link
Copy Markdown
Member

@sethmlarson sethmlarson commented Nov 11, 2020

Previously if b"user-agent" was sent for a non-chunked request then we'd emit multiple User-Agent headers, this closes that gap. This commit will need to be backported to 1.26

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 11, 2020

Codecov Report

Merging #2047 (ebe7051) into master (00f1769) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2047   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           25        25           
  Lines         2297      2297           
=========================================
  Hits          2297      2297           
Impacted Files Coverage Δ
src/urllib3/connection.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00f1769...ebe7051. Read the comment docs.

@sethmlarson sethmlarson added this to the v1.26 milestone Nov 11, 2020
@sethmlarson sethmlarson merged commit 74d6be1 into urllib3:master Nov 11, 2020
@sethmlarson sethmlarson deleted the user-agent-sent-twice branch November 11, 2020 18:51
This was referenced Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants