[Tsv-art] Review of draft-hardie-privsec-metadata-insertion-05

=?utf-8?q?Michael_T=C3=BCxen_=3Ctuexen=40fh-muenster=2Ede=3E?=@ietfa.amsl.com Thu, 16 February 2017 18:18 UTC

Return-Path: <tuexen@fh-muenster.de>
X-Original-To: tsv-art@ietf.org
Delivered-To: tsv-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2AB241294F5; Thu, 16 Feb 2017 10:18:28 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: "Michael Tüxen <tuexen@fh-muenster.de>"@ietfa.amsl.com
To: tsv-art@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.44.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148726910816.1075.17105591789960030239.idtracker@ietfa.amsl.com>
Date: Thu, 16 Feb 2017 10:18:28 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/BZT0BkgbmQB5uY9msd2we6MX36w>
X-Mailman-Approved-At: Fri, 17 Feb 2017 09:52:11 -0800
Cc: ietf@ietf.org, draft-hardie-privsec-metadata-insertion.all@ietf.org
Subject: [Tsv-art] Review of draft-hardie-privsec-metadata-insertion-05
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.17
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Feb 2017 18:18:28 -0000

Reviewer: Michael Tüxen
Review result: Ready with Nits

I've reviewed this document as part of the transport area
directorate's ongoing
effort to review key IETF documents.
These comments were written primarily for the transport area
directors,
but are copied to the document's authors for their information and to
allow
them to address any issues raised. When done at the time of IETF Last
Call,
the authors should consider this review together with any other
last-call
comments they receive.
Please always CC tsv-art@ietf.org if you reply to or forward this
review.

This draft is basically ready for publication, but has nits that
should be
fixed before publication.

Running https://tools.ietf.org/tools/idnits/idnits.pyht reports
* The abstract seems to contain references ([RFC7624]), which it
shouldn't.
* Unused Reference: 'RFC4301'

The discussion of RFC 7871 in section 4 could be improved to allow
readers
not knowing the EDNS0 option to get the point. Either provide an
abstract
description or refer to RFC 6891.