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)

x86_64
Linux
defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: ohtsu, Assigned: ohtsu)

Details

(Whiteboard: [spdy])

Attachments

(1 file)

Attached patch mypatch.diffSplinter 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.
thanks..
Assignee: nobody → ohtsu
Whiteboard: [spdy]
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.
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.
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.