[Gen-art] Gen-ART review of draft-ietf-geopriv-dhcp-lbyr-uri-option-15

Alexey Melnikov <alexey.melnikov@isode.com> Sat, 09 June 2012 16:47 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7D07021F87D1; Sat, 9 Jun 2012 09:47:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.856
X-Spam-Level:
X-Spam-Status: No, score=-102.856 tagged_above=-999 required=5 tests=[AWL=-0.257, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id GJJOAuutpOr5; Sat, 9 Jun 2012 09:47:26 -0700 (PDT)
Received: from rufus.isode.com (cl-125.lon-03.gb.sixxs.net [IPv6:2a00:14f0:e000:7c::2]) by ietfa.amsl.com (Postfix) with ESMTP id 87FB121F8783; Sat, 9 Jun 2012 09:47:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1339260445; d=isode.com; s=selector; i=@isode.com; bh=jp71fRvKlk5rYaTwrHr99GBIQX/cibP+V7SWi3RV3tc=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=AAElZeFeFplyo8Ls4bnsuSNV262nc8DG4qUEMQ3VNzhm8ywFZo89oIy44QIqkFfpHsVHsH 7zneF7GiY9oVJaDWkIMlhJSsFCY+2PGrc3fErCN6OWMFGGIKL06ETzS2eN/4nRbq+/z5XA dsa9qDufSg2+IzQx9Ec2cDIfmkIJFxM=;
Received: from [172.16.1.29] (shiny.isode.com [62.3.217.250]) by rufus.isode.com (submission channel) via TCP with ESMTPSA id <T9N-HAAE47Bf@rufus.isode.com>; Sat, 9 Jun 2012 17:47:25 +0100
X-SMTP-Protocol-Errors: PIPELINING
Message-ID: <4FD37E36.9080203@isode.com>
Date: Sat, 09 Jun 2012 17:47:50 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:10.0.2) Gecko/20120216 Thunderbird/10.0.2
To: draft-ietf-geopriv-dhcp-lbyr-uri-option.all@tools.ietf.org
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: gen-art@ietf.org, The IESG <iesg@ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-geopriv-dhcp-lbyr-uri-option-15
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 09 Jun 2012 16:47:27 -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>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-geopriv-dhcp-lbyr-uri-option-15
Reviewer: Alexey Melnikov
Review Date: 2012-06-09
IETF LC End Date: 2012-06-14
IESG Telechat date: (if known)

Summary: This document is nearly ready for publication as a Proposed 
Standard.

Major issues: None

Minor issues:

In Section 1: the first reference to URI needs a Normative reference. 
The whole document is about passing URIs and they are not defined anywhere.

In Section 2.3: LuriType =1 description says that the value is UTF-8. 
This is not correct, URIs never contain non-ASCII values. So please 
either say ASCII, or use IRI if you really meant UTF-8 (but use of IRIs 
would be horrible, so don't).

LuriType=2: in network byte order?

In Section 3.2: "data:" URI needs an Informative reference (RFC 2397).

In Section 3.3: all media types mentioned (two?) need Normative references.


Nits/editorial comments:

In Section 3, 2nd to the last para: s/deference/dereference ?