[Gen-art] Gen-ART Telechat Review of draft-bryan-metalinkhttp-21

Ben Campbell <ben@estacado.net> Tue, 01 March 2011 22:10 UTC

Return-Path: <ben@estacado.net>
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 286543A6B00 for <gen-art@core3.amsl.com>; Tue, 1 Mar 2011 14:10:30 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.424
X-Spam-Level:
X-Spam-Status: No, score=-2.424 tagged_above=-999 required=5 tests=[AWL=0.175, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id syQpmZR3plhr for <gen-art@core3.amsl.com>; Tue, 1 Mar 2011 14:10:29 -0800 (PST)
Received: from estacado.net (estacado-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:266::2]) by core3.amsl.com (Postfix) with ESMTP id B84343A6B04 for <gen-art@ietf.org>; Tue, 1 Mar 2011 14:10:28 -0800 (PST)
Received: from [10.0.1.6] (cpe-76-183-178-106.tx.res.rr.com [76.183.178.106]) (authenticated bits=0) by estacado.net (8.14.3/8.14.2) with ESMTP id p21MBMUN089968 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Tue, 1 Mar 2011 16:11:27 -0600 (CST) (envelope-from ben@estacado.net)
From: Ben Campbell <ben@estacado.net>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Date: Tue, 01 Mar 2011 16:11:22 -0600
Message-Id: <82C4B08B-587B-4518-8275-AC424313E87B@estacado.net>
To: draft-bryan-metalinkhttp.all@tools.ietf.org, General Area Review Team <gen-art@ietf.org>
Mime-Version: 1.0 (Apple Message framework v1082)
X-Mailer: Apple Mail (2.1082)
Subject: [Gen-art] Gen-ART Telechat Review of draft-bryan-metalinkhttp-21
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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: Tue, 01 Mar 2011 22:10:30 -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 wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-bryan-metalinkhttp-21
Reviewer: Ben Campbell
Review Date:  2011-03-01
IETF LC End Date:
IESG Telechat date: 2011-03-03

Summary: This draft is essentially ready for publication as a proposed standard. I have a couple of remaining comments that may be worth considering, but probably should not block the draft's progress.

Major issues:

None.

Minor issues:

-- I note that the draft still suggests that servers SHOULD share an eTag generation policy, but does not offer a baseline policy that servers either SHOULD or MUST implement. I still have a mild concern that implementations might have non-intersecting sets of policy options. On the other hand, I realize that coming up with (i.e. agreeing to) a baseline policy is not a trivial effort in itself, so I don't think this should block publication. I mention it merely for the sake of completeness.

Nits/editorial comments:

-- section 2, last paragraph "Metalink clients use the mirrors provided by a Metalink server in Link header fields [ RFC5988 ] but it is restricted to the initial Metalink server they contacted. "

s /"... it is restricted..." / "... this is restricted ..."

Also, is there a risk the initial server could cause a loop by pointing to _itself_?