Skip to content

Follow up tweaks to Host header precedence for :authority in HTTP/2 - #243

Merged
lovelydinosaur merged 1 commit into
masterfrom
follow-up-to-host-header-precedence-for-http2
Nov 27, 2020
Merged

lovelydinosaur merged 1 commit into
masterfrom
follow-up-to-host-header-precedence-for-http2

Conversation

@lovelydinosaur

Copy link
Copy Markdown
Member

Follow up to #241

  • Drop pseudo-headers being stripped. Let's treat any behaviour there separately.
  • If a Host header is included, then don't auto-append port info onto it.
  • Minor tweak of comment.

@florimondmanca florimondmanca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense!

@lovelydinosaur
lovelydinosaur merged commit 641a762 into master Nov 27, 2020
@lovelydinosaur
lovelydinosaur deleted the follow-up-to-host-header-precedence-for-http2 branch November 27, 2020 13:41
@florimondmanca florimondmanca mentioned this pull request Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants