[Gen-art] Genart last call review of draft-ietf-httpbis-bcp56bis-13

David Schinazi via Datatracker <noreply@ietf.org> Thu, 12 August 2021 18:36 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 424103A45C8; Thu, 12 Aug 2021 11:36:09 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: David Schinazi via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-httpbis-bcp56bis.all@ietf.org, ietf-http-wg@w3.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.36.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <162879336918.10574.10010152000496998787@ietfa.amsl.com>
Reply-To: David Schinazi <dschinazi.ietf@gmail.com>
Date: Thu, 12 Aug 2021 11:36:09 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/qVJ9Vf6SAeh7nATa3apYtca8P8w>
Subject: [Gen-art] Genart last call review of draft-ietf-httpbis-bcp56bis-13
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: Thu, 12 Aug 2021 18:36:10 -0000

Reviewer: David Schinazi
Review result: Ready with Issues

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-bcp56bis-13
Reviewer: David Schinazi
Review Date: 2021-08-12
IETF LC End Date: 2021-07-23
IESG Telechat date: Not scheduled for a telechat

Summary: Well-written and easy to read document.

Major issues: None

Minor issues:
* s4.5 seems to prohibit defining new non-generic HTTP methods. How do we
reconcile that with the work happening in MASQUE? I know that CONNECT is its
own special-case, but should we have a carveout here? (Though MASQUE might end
up using extended CONNECT which side steps the issue). Or is it the case that
MASQUE is modifying HTTP itself instead of building an application over HTTP?

Nits/editorial comments:
* s3.2 uses the term "link" without explaining what it is. Perhaps a reference
to RFC 8288 if that's what is meant here? * s4.11 mentions HTTP/3 without
referencing its specification