Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Martin Duke's QPACK comments #4750

Merged
merged 3 commits into from Jan 25, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 6 additions & 5 deletions draft-ietf-quic-qpack.md
Expand Up @@ -471,9 +471,10 @@ this MUST be treated as a connection error of type QPACK_ENCODER_STREAM_ERROR.
## Dynamic Table {#header-table-dynamic}

The dynamic table consists of a list of field lines maintained in first-in,
first-out order. Each HTTP/3 endpoint holds a dynamic table that is initially
empty. Entries are added by encoder instructions received on the encoder
stream; see {{encoder-instructions}}.
first-out order. A QPACK encoder and decoder share a dynamic table that is
initially empty. The encoder adds entries to the dynamic table and sends them
to the decoder via instructions on the encoder stream; see
{{encoder-instructions}}.

The dynamic table can contain duplicate entries (i.e., entries with the same
name and same value). Therefore, duplicate entries MUST NOT be treated as an
Expand All @@ -486,8 +487,8 @@ Dynamic table entries can have empty values.
The size of the dynamic table is the sum of the size of its entries.

The size of an entry is the sum of its name's length in bytes, its value's
length in bytes, and 32. The size of an entry is calculated using the length of
its name and value without Huffman encoding applied.
length in bytes, and 32 additional bytes. The size of an entry is calculated
using the length of its name and value without Huffman encoding applied.

### Dynamic Table Capacity and Eviction {#eviction}

Expand Down