- From: Martin Duke via Datatracker <noreply@ietf.org>
- Date: Sun, 17 May 2020 15:21:59 -0700
- To: "The IESG" <iesg@ietf.org>
- Cc: draft-ietf-httpbis-header-structure@ietf.org, httpbis-chairs@ietf.org, ietf-http-wg@w3.org, Tommy Pauly <tpauly@apple.com>, tpauly@apple.com
Martin Duke has entered the following ballot position for draft-ietf-httpbis-header-structure-18: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-httpbis-header-structure/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- This is probably a simple one, and perhaps I'm missing something obvious: Throughout Section 3, the document specifies minimum data structure sizes (1024 list members, 256 inner list members, 64-character keys, etc.) that the receiver MUST be able to process. What is the desired behavior if any of these data structures exceeds what the receiver can process? Must it skip the entire field, or can it process the first N entries and then ignore the rest? Given the "Intentionally Strict Processing" principle, it would be good to spell this out. ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thanks for this noble attempt to tame the wildness that is the HTTP spec! Comments: - While this is by no means a required change to publish this document, I found the order of Section 3 to be backwards from what would easiest to follow. The higher-order concepts (e.g. lists) are defined first, and refer to low-level concepts (like items) that are not defined till the end of the section. Nits: - In Sec 3.1.2, it might be useful to explain that in example-IntHeader, a is TRUE. - sec 3.2. Can you add some text to make it clear that the value in dictionary entries is only optional (in brackets) because of Boolean TRUE? This was not clear to me until I read sec. 4.1.2. - Sec 4. s/before HPACK is applied/before compression with HPACK (A receiver "applies" HPACK to decompress, and presumably before doing this parsing) - Sec 4.2. s/header value/field value
Received on Sunday, 17 May 2020 22:22:14 UTC