http/2 doesn't send HPACK-level ack of table size changes

RESOLVED FIXED in mozilla32

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: u408661, Assigned: u408661)

Tracking

unspecified
mozilla32
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [spdy])

Attachments

(1 attachment)

Assignee

Description

5 years ago
Posted patch patchSplinter Review
I wondered why Herve added that to the HPACK spec, seemed like a duplication of effort, to me. But (b/c of Jeff's emails), Martin explained it to me yesterday, so I figured I'd fix it. This patch was built on top of the draft12, but it applies (conceivably) to draft10, which is currently in nightly. I vote we just land draft11, draft12, and this all together.
Attachment #8414630 - Flags: review?(mcmanus)
Comment on attachment 8414630 [details] [diff] [review]
patch

Review of attachment 8414630 [details] [diff] [review]:
-----------------------------------------------------------------

cset comment s/SETTINGS_MAX_BUFFER_SIZE/SETTINGS_HEADER_TABLE_SIZE/
Attachment #8414630 - Flags: review?(mcmanus) → review+
Depends on: 1001022
Assignee

Comment 2

5 years ago
Thanks, Pat! Got a couple errands to run, then a quick interop with twitter, and then I'll land draft11+draft12+this later tonight (PDT).
:hurley, I've just been running the latest (as of this minute) stuff from your draft-12 against twitter.com. It just works! I've run curl+nghttp2 against twitter with the same success, so there's that level of interop at least...
Assignee

Comment 4

5 years ago
:bagder - good to know. I'll still want to test with the changes I'm making based in review comments, as well as this patch (it's in a different branch from draft12) to be sure, but your report is a promising sign!
https://hg.mozilla.org/mozilla-central/rev/68e78cfc9094
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.