Closed Bug 1508709 Opened 3 years ago Closed 3 years ago

HPACK, encode path=/ with TABLE_INDEX "4"

Categories

(Core :: Networking: HTTP, defect)

65 Branch
defect
Not set
normal

Tracking

()

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: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.