Gen-ART Review of draft-ietf-tzdist-service-08

Russ Housley <housley@vigilsec.com> Fri, 05 June 2015 20:09 UTC

Return-Path: <housley@vigilsec.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 57D7A1B319D; Fri, 5 Jun 2015 13:09:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.9
X-Spam-Level:
X-Spam-Status: No, score=-101.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, USER_IN_WHITELIST=-100] autolearn=ham
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 Clb_QE78nxLD; Fri, 5 Jun 2015 13:09:58 -0700 (PDT)
Received: from odin.smetech.net (x-bolt-wan.smeinc.net [209.135.219.146]) by ietfa.amsl.com (Postfix) with ESMTP id CCDB31B313F; Fri, 5 Jun 2015 13:09:57 -0700 (PDT)
Received: from localhost (unknown [209.135.209.5]) by odin.smetech.net (Postfix) with ESMTP id 7F0539A4064; Fri, 5 Jun 2015 16:09:47 -0400 (EDT)
X-Virus-Scanned: amavisd-new at smetech.net
Received: from odin.smetech.net ([209.135.209.4]) by localhost (ronin.smeinc.net [209.135.209.5]) (amavisd-new, port 10024) with ESMTP id zhVptkTJDz7j; Fri, 5 Jun 2015 16:08:27 -0400 (EDT)
Received: from [192.168.2.100] (pool-108-51-128-219.washdc.fios.verizon.net [108.51.128.219]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by odin.smetech.net (Postfix) with ESMTP id A2F5A9A400D; Fri, 5 Jun 2015 16:09:26 -0400 (EDT)
From: Russ Housley <housley@vigilsec.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Subject: Gen-ART Review of draft-ietf-tzdist-service-08
Date: Fri, 05 Jun 2015 16:09:26 -0400
Message-Id: <0ABAD05A-157F-4459-A9E1-E8EFB78B8413@vigilsec.com>
To: draft-ietf-tzdist-service.all@ietf.org
Mime-Version: 1.0 (Apple Message framework v1085)
X-Mailer: Apple Mail (2.1085)
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/c3hiI9ts20at7UHQCiZAaFFiJO4>
Cc: IETF Gen-ART <gen-art@ietf.org>, IETF <ietf@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.15
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: <http://www.ietf.org/mail-archive/web/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: Fri, 05 Jun 2015 20:09:59 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

This review is in response to a request for early Gen-ART review.

Document: draft-ietf-tzdist-service-08
Reviewer: Russ Housley
Review Date: 2015-06-05
IETF LC End Date: 2015-06-17
IESG Telechat date: unknown

Summary: Almost Ready


Major Concerns:

In section 5.6, it is not clear to me how to distinguish the addition
of a leap second from the removal of a leap second.  The UTC offset
from TAI in seconds is provided.  And, so far, we have never seen a
negative leap second.  Is the assumption that we will never see so
many negative ones that the offset is les than zero?  Please clarify.


Minor Concerns:

Section 4.2.1.3 says: 'The "well-known" URI is always present on the
server, even when a TXT RR (Section 4.2.1.2) is used in the DNS to
specify a "context path".'  I think it would be better to reword this
as a MUST statement.

Section 10.1.1 says: "Decisions made by the designated expert can be
appealed to the IESG Applications Area Director, then to the IESG."
The IESG just merged the Applications Area and the RAI Area, creating
the ART Area.  Is there a way to word this that can avoid confusion
when the IESG makes further organizational changes?

Section 10.2 says: "Change controller:  IETF."  Would it be better for
the IESG to be the change controller?  This provides better alignment
with Section 10.3.

Section 10.2 incudes a heading for "Related information".  Something
needs to go here.  If there is nothing to add, then say "None."


Other Comments:

The are places in the document where there are many blank lies at the
botton of the page.  I'm sure the RFC Editor can fix them, but if you
need to spin a new version, then you might address that too.

Section 4.1.4 says: "If a client only needs data for only one, ..."
There is an extra "only", please drop the first one.

Section 4.2.1.2 says: "... URI approach described next."  However, the
description is a few paragraphs away.  It might be better to say that
the approach is described in the next section, or even give the section
upcoming number.

In the IANA Considerations Section, this document is referenced at least
three ways: RFCXXXX, This RFC, and This Draft.  Please pick one.