Closed Bug 1508709 Opened 7 years ago Closed 7 years ago

HPACK, encode path=/ with TABLE_INDEX "4"

Categories

(Core :: Networking: HTTP, defect)

Product:
Core
Component:
Networking: HTTP
Version:
65 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: andreas.filsinger, Unassigned)

Details

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:65.0) Gecko/20100101 Firefox/65.0 Steps to reproduce: initiate firefox to do a very first request to a HTTP/2 Server, and inspect the generated h2 client-(request)-encoding. decode the HEADER_FRAME, look at the HEADER-Data. This is all about the representation of ":path=/" in the ever first request header. Actual results: firefox encodes "path=/" as 0|0|0|0|TABLE_INDEX(4) 1|<LEN> Huffman-Encode("/") Expected results: firefox should encode this like: 1|TABLE_INDEX(4) a) compressing a single char "/" into huffman is no win b) sending a value that already lives in the static Table is no win
Setting component as Core: Networking HTTP. Please correct if this is not the right component. Thanks.
Component: Untriaged → Networking: HTTP
Product: Firefox → Core
Per [1], we intentionally add this to the static table [2] [1] https://httpwg.org/specs/rfc7541.html#static.table.entries [2] https://searchfox.org/mozilla-central/rev/0859e6b10fb901875c80de8f8fc33cbb77b2505e/netwerk/protocol/http/Http2Compression.cpp#125
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.