Skip to content

Commit

Permalink
Clarify when to signal error on incorrect Required Insert Count.
Browse files Browse the repository at this point in the history
  • Loading branch information
bencebeky committed Jan 23, 2019
1 parent 3aa0825 commit b24b3ee
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions draft-ietf-quic-qpack.md
Expand Up @@ -253,7 +253,16 @@ received.
Each header block contains a Required Insert Count, the lowest possible value
for the Insert Count with which the header block can be decoded. For a header
block with no references to the dynamic table, the Required Insert Count is
zero.
zero. For a header block with references to the dynamic table, the Required
Insert Count is one larger than the largest Absolute Index of all referenced
dynamic table entries.

If the decoder encounters a header block with a Required Insert Count value
larger than defined here, it MAY treat this as a stream error of type
HTTP_QPACK_DECOMPRESSION_FAILED. If the decoder encounters a header block with
a Required Insert Count value smaller than defined here, it MUST treat this as a
stream error of type HTTP_QPACK_DECOMPRESSION_FAILED as prescribed in
{{invalid-references}}.

When the Required Insert Count is zero, the frame contains no references to the
dynamic table and can always be processed immediately.
Expand All @@ -265,10 +274,6 @@ unblocked when the Insert Count becomes greater than or equal to the Required
Insert Count for all header blocks the decoder has started reading from the
stream.

If the decoder encounters a header block where the largest Absolute Index used
is not equal to the largest value permitted by the Required Insert Count, it MAY
treat this as a stream error of type HTTP_QPACK_DECOMPRESSION_FAILED.

The SETTINGS_QPACK_BLOCKED_STREAMS setting (see {{configuration}}) specifies an
upper bound on the number of streams which can be blocked. An encoder MUST limit
the number of streams which could become blocked to the value of
Expand Down

0 comments on commit b24b3ee

Please sign in to comment.