Created attachment 8529098 [details] [diff] [review] mypatch.diff 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.
Assignee: nobody → ohtsu
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.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.