Re: [Gen-art] Gen-ART LC review of draft-ietf-dnsop-cookies-08

Donald Eastlake <> Mon, 28 December 2015 00:31 UTC

Return-Path: <>
Received: from localhost ( []) by (Postfix) with ESMTP id E451E1A86E4; Sun, 27 Dec 2015 16:31:22 -0800 (PST)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: 0.55
X-Spam-Status: No, score=0.55 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, MANGLED_LIST=2.3, SPF_PASS=-0.001] autolearn=no
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id WRXd51pULamK; Sun, 27 Dec 2015 16:31:20 -0800 (PST)
Received: from ( [IPv6:2607:f8b0:4003:c01::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 9ABEC1A86E3; Sun, 27 Dec 2015 16:31:20 -0800 (PST)
Received: by with SMTP id iw8so223340024obc.1; Sun, 27 Dec 2015 16:31:20 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=MDDkPDsC1iKtZeH4/d/0UupRjsX4YEiO2lXtOkEwMz0=; b=kniSccobf3Ha6jwclRy9Ee0wAL3J6NzNF0MNLCB2T2wdX2Kme5a3rBymBlY2K1jqI+ xt1xU/anzi47luDZmKl018kPOsp7X61E4SbTotNc4OSUNFBWFHQyJEcrGEItdN0870e8 uvnfI6HRTYEXjT+cm0K7VowF/g4jbOv/mqqTxhc0+KbmOcqqiosQ9Dd0JAKdJI86XGK4 oHtB+bvaSzL6nYD7f3ntlhvA1DTudeA3R5rlFHngOc09q/vD0niiro1wMsu72rEQM8QW QUaRZFA/mVULVHoYT8mmuTlX7EyyECi8c+/T9pfQ6YF17oOPthTNhmcLywpo4nztYHlg 6G4A==
X-Received: by with SMTP id p12mr31952834oeu.55.1451262679949; Sun, 27 Dec 2015 16:31:19 -0800 (PST)
MIME-Version: 1.0
Received: by with HTTP; Sun, 27 Dec 2015 16:31:05 -0800 (PST)
In-Reply-To: <011001d13eb3$63339cd0$299ad670$>
References: <011001d13eb3$63339cd0$299ad670$>
From: Donald Eastlake <>
Date: Sun, 27 Dec 2015 19:31:05 -0500
Message-ID: <>
To: Peter Yee <>
Content-Type: text/plain; charset=UTF-8
Archived-At: <>
Cc: " Review Team" <>,, IETF Discussion <>
Subject: Re: [Gen-art] Gen-ART LC review of draft-ietf-dnsop-cookies-08
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Mon, 28 Dec 2015 00:31:23 -0000

Hi Peter,

Thanks for the comments, see below.

On Thu, Dec 24, 2015 at 8:27 PM, Peter Yee <> wrote:
> 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
> comment.  For background on Gen-ART, please see the FAQ at
> <>
> (Actually, I'm tardy on this review.  It inexplicably dropped off my radar.
> So deal with these comments when you get around to handling Telechat input
> or AUTH48 or whenever it suits you!  I'm still posting this review as it
> will be needed come the Telechat.)

I don't think it has been scheduled for a Telechat yet so we might as
well resolve these comments now.

> Document: draft-ietf-dnsop-cookies-08
> Reviewer: Peter Yee
> Review Date: December 24, 2015
> IETF LC End Date: December 14, 2015
> IESG Telechat date: TBD
> Summary: This draft is basically ready for publication, but has nits that
> should be fixed before publication. [Ready with nits]
> The draft provides a lightweight means to increase the difficulty of certain
> DNS attacks by off-path attackers, but it isn't designed to be the be all
> and end all of DNS security.  It can be deployed incrementally.
> Major issues: None
> Minor issues:
> Page 14, Section 5.2.4, 1st paragraph, 1st sentence: It might be useful to
> mention what the examination entails as it would help in understanding the
> 3rd sentence in the paragraph.  There's an implied recalculation of the
> Server Cookie value based on the received Client Cookie and client IP
> address as opposed to a simple lookup of the received value.

I'm not so sure of that. If the server wanted to, it could generate a
random Server Cookie for each {Client Cookie, Client IP} and, in fact,
do a look up.

> Nits:
> Page 12, Section 5.2, 3rd paragraph, 1st sentence: change "the the" to just
> "the".


> Page 13, Section 5.2.2, 2nd paragraph: append "bytes" after "40".

Why after 40 but not after 8 or 16? Seems like me it would be an
improvement to add "bytes" after all three.

> Page 14, Section 5.2.4, 1st paragraph, 2nd sentence: delete the sentence.
> It's redundant with the 1st sentence.


> Page 15, Section 5.4, 2nd paragraph, 1st sentence: change first "a" to "an".


> Page 15, Section 5.4, 4th paragraph, 1st sentence: change first "a" to "an".


> Page 17, Section 6, 1st paragraph, 2nd sentence: change "indefinitely" to
> "indefinite".


> Page 21, Section 9, 2nd paragraph, 2nd sentence: change "WPAv2" to "WPA2"
> (the Wi-Fi Alliance's term).


> Page 23, Section 10: change "a" to "an".


> Page 27, Section A.1, 1st sentence: change "An" to "A".


> Page 29, 1st partial sentence: if you're going to drop beta earlier in the
> section, you might as well give the BIND version number here as well.  It's
> no longer apparent that a beta version was involved.


 Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
 155 Beaver Street, Milford, MA 01757 USA