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

Michael Tuexen <tuexen@fh-muenster.de> Thu, 23 February 2017 13:58 UTC

Return-Path: <tuexen@fh-muenster.de>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4046C12940F; Thu, 23 Feb 2017 05:58:48 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.6
X-Spam-Level:
X-Spam-Status: No, score=-2.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kjfsYZcQYwQR; Thu, 23 Feb 2017 05:58:46 -0800 (PST)
Received: from drew.franken.de (drew.ipv6.franken.de [IPv6:2001:638:a02:a001:20e:cff:fe4a:feaa]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D744C1296D2; Thu, 23 Feb 2017 05:58:42 -0800 (PST)
Received: from [IPv6:2003:cd:6be5:a300:487d:7c9d:5944:feda] (p200300CD6BE5A300487D7C9D5944FEDA.dip0.t-ipconnect.de [IPv6:2003:cd:6be5:a300:487d:7c9d:5944:feda]) (Authenticated sender: macmic) by mail-n.franken.de (Postfix) with ESMTPSA id 5E655721E281A; Thu, 23 Feb 2017 14:58:38 +0100 (CET)
From: Michael Tuexen <tuexen@fh-muenster.de>
Message-Id: <792C410E-98F1-42E9-84C8-FC71BC8528F9@fh-muenster.de>
Content-Type: multipart/signed; boundary="Apple-Mail=_A8E529D9-0FC5-4052-A2AA-928B22F5C8A4"; protocol="application/pkcs7-signature"; micalg="sha1"
Mime-Version: 1.0 (Mac OS X Mail 10.2 \(3259\))
Subject: Re: [Tsv-art] Review of draft-hardie-privsec-metadata-insertion-05
Date: Thu, 23 Feb 2017 14:58:37 +0100
In-Reply-To: <CA+9kkMAmG2P=g1k7w3YOh1udjDB_MOjn0=+hnUXap1zq-VtwHQ@mail.gmail.com>
To: Ted Hardie <ted.ietf@gmail.com>
References: <148726910816.1075.17105591789960030239.idtracker@ietfa.amsl.com> <CA+9kkMAmG2P=g1k7w3YOh1udjDB_MOjn0=+hnUXap1zq-VtwHQ@mail.gmail.com>
X-Mailer: Apple Mail (2.3259)
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/nVdwgEXoafuX9-SDHGgRy3ryio0>
Cc: <>, "Michael TXXxen <tuexen@fh-muenster.de>"@ietfa.amsl.com, IETF <ietf@ietf.org>, draft-hardie-privsec-metadata-insertion.all@ietf.org, tsv-art@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
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: Thu, 23 Feb 2017 13:58:48 -0000

> On 22 Feb 2017, at 22:53, Ted Hardie <ted.ietf@gmail.com> wrote:
> 
> Hi Michael,
> 
> Thanks for the review; I've uploaded a -06 now which updates the abstract, adds the reference to RFC 6891 and clarifies the language around the purpose of the EDNS0 options.
Thanks for the update. It addresses the issues I found.

Best regards
Michael
> 
> regards,
> 
> Ted
> 
> On Thu, Feb 16, 2017 at 10:18 AM, <"Michael Tüxen <tuexen@fh-muenster.de>"@ietfa.amsl.com> wrote:
> 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.
> 
> 
> 
> _______________________________________________
> Tsv-art mailing list
> Tsv-art@ietf.org
> https://www.ietf.org/mailman/listinfo/tsv-art