[apps-discuss] apps-team review of draft-ietf-sidr-ghostbusters-09
Barry Leiba <barryleiba@computer.org> Tue, 13 September 2011 17:15 UTC
Return-Path: <barryleiba@gmail.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D12AE21F8C51; Tue, 13 Sep 2011 10:15:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.007
X-Spam-Level:
X-Spam-Status: No, score=-103.007 tagged_above=-999 required=5 tests=[AWL=-0.030, BAYES_00=-2.599, FM_FORGED_GMAIL=0.622, RCVD_IN_DNSWL_LOW=-1, 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 nallJSQ3T8re; Tue, 13 Sep 2011 10:15:47 -0700 (PDT)
Received: from mail-gy0-f172.google.com (mail-gy0-f172.google.com [209.85.160.172]) by ietfa.amsl.com (Postfix) with ESMTP id DA1B421F8C4E; Tue, 13 Sep 2011 10:15:46 -0700 (PDT)
Received: by gyd12 with SMTP id 12so727665gyd.31 for <multiple recipients>; Tue, 13 Sep 2011 10:17:53 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:cc:content-type; bh=OVUc9CQyoPnmNiJyLEG99nuJ4uN/Xa3oc+f1kXQ8c1w=; b=FfxHCBuBPlaLVXD+nMVEVRN2MWCJA8TjQw2wAY9BPBv+Zg14tqedjbVEoXbk4TLmPc k06z90dAlh1hlG7CLGiGIzynur7mriYz7HOsdM4PWKFK1uKdicjeanCkR413xlwdy7nA 0B3GgpSEZXG1zRCwkbI3brNe5OQt7HD3gB6yc=
MIME-Version: 1.0
Received: by 10.236.145.10 with SMTP id o10mr35810021yhj.90.1315934273391; Tue, 13 Sep 2011 10:17:53 -0700 (PDT)
Sender: barryleiba@gmail.com
Received: by 10.236.203.68 with HTTP; Tue, 13 Sep 2011 10:17:53 -0700 (PDT)
Date: Tue, 13 Sep 2011 13:17:53 -0400
X-Google-Sender-Auth: jfkYQfWQh92cqNa1RJmZf4BQb0Y
Message-ID: <CALaySJJfu8T6QZ2fQfAwUL32hgOG9kPkioPO+tQZZLNGf12HNQ@mail.gmail.com>
From: Barry Leiba <barryleiba@computer.org>
To: apps-discuss@ietf.org, draft-ietf-sidr-ghostbusters.all@tools.ietf.org
Content-Type: text/plain; charset="ISO-8859-1"
Cc: iesg@ietf.org
Subject: [apps-discuss] apps-team review of draft-ietf-sidr-ghostbusters-09
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 13 Sep 2011 17:15:48 -0000
I have been selected as the Applications Area Review Team reviewer for this draft (for background on apps-review, please see http://www.apps.ietf.org/content/applications-area-review-team ). Please resolve these comments along with any other Last Call comments you may receive. Please wait for direction from your document shepherd or AD before posting a new version of the draft. Document: draft-ietf-sidr-ghostbusters-09 Title: The RPKI Ghostbusters Record Summary: This draft is ready for publication as a Proposed Standard, with just some very minor tweaking requested. -------------------- Major Issues: Section 1, paragraph 2: OLD lead to the worrisome certificate's or CRL's maintainer. So, "Who do you call?" NEW lead to the worrisome certificate's or CRL's maintainer. So, "Who you gonna call?" REFERENCE: http://www.youtube.com/watch?v=m9We2XsVZfc -------------------- Minor Issues: vCard does use the term "type", unfortunately. The problem is that a "type" in vCard is generally something different, and in the RFC we missed some instances of inconsistent use. But the proper vCard term for what you want here is "property". So: Section 3: OLD An example of an RPKI Ghostbusters Record payload with all types populated is as follows: NEW An example of an RPKI Ghostbusters Record payload with all properties populated is as follows: Section 4: OLD The goal in profiling the vCARD is not to include as much information as possible, but rather to include as few types as possible while NEW The goal in profiling the vCard is not to include as much information as possible, but rather to include as few properties as possible while OLD Per [RFC6350], the BEGIN, VERSION, FN, N, and END types MUST be included in a record. To be useful, one or more of ADR, TEL, and EMAIL MUST be included. Other types MUST NOT be included. NEW Per [RFC6350], the BEGIN, VERSION, FN, N, and END properties MUST be included in a record. To be useful, one or more of ADR, TEL, and EMAIL MUST be included. Other properties MUST NOT be included. -------------------- Nits: Throughout: RFC 6350 uses "vCard", while this document uses "vCARD". It's not a big thing, but it's easy to be consistent with 6350 and switch to "vCard". Section 7: OLD Though there is no on the wire protocol in this specification, NEW Though there is no on-the-wire protocol in this specification, -------------------- Barry
- [apps-discuss] apps-team review of draft-ietf-sid… Barry Leiba
- Re: [apps-discuss] apps-team review of draft-ietf… Cyrus Daboo
- Re: [apps-discuss] apps-team review of draft-ietf… Barry Leiba
- Re: [apps-discuss] apps-team review of draft-ietf… Barry Leiba
- Re: [apps-discuss] apps-team review of draft-ietf… Barry Leiba
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Simon Perreault
- Re: [apps-discuss] apps-team review of draft-ietf… Murray S. Kucherawy
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… SM
- Re: [apps-discuss] apps-team review of draft-ietf… Randy Bush
- Re: [apps-discuss] apps-team review of draft-ietf… Simon Perreault
- Re: [apps-discuss] apps-team review of draft-ietf… Sandra Murphy