[Gen-art] Genart last call review of draft-ietf-httpbis-cache-16

Mohit Sethi via Datatracker <noreply@ietf.org> Sun, 30 May 2021 14:14 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2DAC13A3DAE; Sun, 30 May 2021 07:14:33 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Mohit Sethi via Datatracker <noreply@ietf.org>
To: <gen-art@ietf.org>
Cc: draft-ietf-httpbis-cache.all@ietf.org, ietf-http-wg@w3.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.30.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <162238407309.22812.14001073203740035939@ietfa.amsl.com>
Reply-To: Mohit Sethi <mohit.m.sethi@ericsson.com>
Date: Sun, 30 May 2021 07:14:33 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/YWMvuhly5ShJtk5PJ09KCLw-0vk>
Subject: [Gen-art] Genart last call review of draft-ietf-httpbis-cache-16
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 30 May 2021 14:14:33 -0000

Reviewer: Mohit Sethi
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-httpbis-cache-16
Reviewer: Mohit Sethi
Review Date: 2021-05-30
IETF LC End Date: 2021-06-10
IESG Telechat date: 2021-06-17

Summary: This draft specification defines HTTP caches and header fields for
controlling the cache behavior.

Major issues:

Minor issues:
- In the HTML version of the draft, the reference to [Semantics] does not work
properly. I looked at the xml source which looks fine. I suspect it is
something to do with the tooling.

- It is not clear to me which draft is creating the "Hypertext Transfer
Protocol (HTTP) Field Name Registry". It seems both this draft and
draft-ietf-httpbis-semantics are creating it? Perhaps you could remove the text
in this draft saying "introduce the new" and just ask IANA to update the
registry with fields in Table 1 of this draft.

Nits/editorial comments:

- Section 1: When does a client or server act as "tunnel"? I don't know if it
is absolutely necessary to explain the term. You can decide.

- Section 1: HTTP caching's goal is significantly improving performance -> HTTP
caching's goal is to significantly improve performance?

- Section 1.3: Maybe it is obvious to many readers, but I was not sure what is
meant by a "canned string"?

- Section 3 vs Section 4: "A cache MUST NOT store a response to a request
unless:" does not have a comma before unless while "When presented with a
request, a cache MUST NOT reuse a stored response, unless:" has a comma before
unless?

- Some of the bullets in section 3 and 4 were hard to parse. Take for example:
"When presented with a request, a cache MUST NOT reuse a stored response,
unless: the stored response does not contain the no-cache cache directive
(Section 5.2.2.4), unless it is successfully validated (Section 4.3), and". I
am not sure how to simplify the text on all these requirements.