Secdir last call review of draft-ietf-httpbis-early-hints-03

Melinda Shore <melinda.shore@gmail.com> Tue, 04 July 2017 19:37 UTC

Return-Path: <melinda.shore@gmail.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 8071D132830; Tue, 4 Jul 2017 12:37:17 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Melinda Shore <melinda.shore@gmail.com>
To: secdir@ietf.org
Cc: draft-ietf-httpbis-early-hints.all@ietf.org, ietf@ietf.org, ietf-http-wg@w3.org
Subject: Secdir last call review of draft-ietf-httpbis-early-hints-03
X-Test-IDTracker: no
X-IETF-IDTracker: 6.55.2
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149919703750.15996.5462759432298024921@ietfa.amsl.com>
Date: Tue, 04 Jul 2017 12:37:17 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/u3DCnx2vlsy97qh3d-NwbU1Fw0U>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.22
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Jul 2017 19:37:18 -0000

Reviewer: Melinda Shore
Review result: Has Issues

I have reviewed this document as part of the security directorate's 
ongoing effort to review all IETF documents being processed by the 
IESG.  These comments were written primarily for the benefit of the 
security area directors.  Document editors and WG chairs should treat 
these comments just like any other last call comments.

Summary:   Has minor issues.

This draft defines a status code for sending an informational 
response that contains header fields that are likely to be included in the 
final response.  A server can send the informational response containing 
some of the header fields to help the client start making preparations 
for processing the final response, and then run time-consuming operations 
to generate the final response.  The informational response can also be used by an
origin server to trigger HTTP/2 server push at a caching intermediary.

Passed nit checker without complaints other than publication date.  Sections
5 and 6 should be appendices.

One minor issue: in the security considerations section, "Therefore, 
a server might refrain from sending Early Hints over HTTP/1.1 unless when 
the client is known to handle informational responses correctly" is a bit squishy
(and contains a superfluous "when").  I'm not sure this merits a text change and
I'm rather certain that it doesn't merit normative 2119 language but it did stand
out as an overly soft recommendation.