Re: [quicwg/base-drafts] Incorrect values for Required Insert Count (#3305)

[afrind <[e-mail]>]

I feel like the draft explicitly says this is an error.  Here's the current text:

> When processing header blocks, the decoder expects the Required Insert Count to exactly match the value defined in Section 2.1.3. If it encounters a smaller value than expected, it MUST treat this as a connection error of type HTTP_QPACK_DECOMPRESSION_FAILED; see Section 2.2.3. If it encounters a larger value than expected, it MAY treat this as a connection error of type HTTP_QPACK_DECOMPRESSION_FAILED.

Which part is obtuse?  Should I replace '...the value defined in Section 2.1.3.'  and '...smaller/larger than expected' with a restatement of the expected values?  I'm pretty sure it used to say that and we changed it to this.  Or would moving it to another section make it easier to find?  

Regarding whether this is a functional or performance bug, I expect many implementations won't parse the header block until the RIC is met, so the encoder can cause the decoder to block a stream indefinitely.  This can lead to other poor compression performance effects, but I don't think it can cause an actual deadlock on any request other than the one with the bad RIC, which is the encoder's fault anyways.  Maybe a security consideration recommending timeouts for this case?  


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/3305#issuecomment-574318635