Closed
Bug 1105302
Opened 9 years ago
Closed 9 years ago
Fix to use indexed encoding for cookies more than 20 bytes
Categories
(Core :: Networking: HTTP, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: ohtsu, Assigned: ohtsu)
Details
(Whiteboard: [spdy])
Attachments
(1 file)
1.32 KB,
patch
|
mcmanus
:
review+
|
Details | Diff | Splinter Review |
I found that cookie headers are always encoded with LiteralHeaderNeverIndexed regardless of their size. The comment in HTTP2Compression.cpp intends that they should be encoded with indexing when their size is more than 20 bytes. There is a written mistake of a variable name in checking a cookie size and this patch corrects it.
Updated•9 years ago
|
Attachment #8529098 -
Flags: review+
Comment 3•9 years ago
|
||
Thank you for identifying and crafting the fix. It has been pushed to gecko 36 which is currently the first branch we announce non-draft h2 support for. If we move the h2 announcement to an earlier branch I'll consider moving this too. I changed a test so that it at least tickles the cookie indexing code.
Assignee | ||
Comment 4•9 years ago
|
||
Thanks for your reviewing and adding the test. I tested it with my iij-http2 server but missed to find where unit tests exist in the Firefox repository. I will take a look at them next time.
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c97ed39af3f7
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•