Re: Gen-art LC review: draft-mcdonald-ipps-uri-scheme-16

Ira McDonald <blueroofmusic@gmail.com> Tue, 25 November 2014 21:27 UTC

Return-Path: <blueroofmusic@gmail.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 2E9EB1A88F7 for <ietf@ietfa.amsl.com>; Tue, 25 Nov 2014 13:27:31 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_PASS=-0.001] 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 Lg40aIjRmo-D for <ietf@ietfa.amsl.com>; Tue, 25 Nov 2014 13:27:28 -0800 (PST)
Received: from mail-wg0-x22f.google.com (mail-wg0-x22f.google.com [IPv6:2a00:1450:400c:c00::22f]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B203F1A6EDA for <ietf@ietf.org>; Tue, 25 Nov 2014 13:27:27 -0800 (PST)
Received: by mail-wg0-f47.google.com with SMTP id n12so2025083wgh.34 for <ietf@ietf.org>; Tue, 25 Nov 2014 13:27:26 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=MGOhmoqfkyBW1qRNY7hm9fX4GYfreNzeRV3PJ9FKcKE=; b=b2cjIeI7OX45pInsRFgIwUxve238YPE4A6Y74g8Eoe/Iq+PkuHObujUVolb4XKG7aF 3FzfjcZB7ZdFn6wcQdQ5IYOiFqIL6B9YuNDcWIihPE4kAfrgyno8JGIEWtx78j9frTvs +74mr4EL79E5KyudVDBh0v6fhdg/jAxlQiOxOZVdQV8I2gR6R7KJpGxd+1z8hsGMmyHU hOlTuTbEHJKmFvFA1qCHhNKbSstENGuL3/rllmU7C+vkh4zIyv0n6cHAALCtITdgOEFZ YTNUUS2osyXFm4ilN+rWWEze/tzZVhhP+iNqobxlm3b5p6Bv5cIzENWMkHPpWaMgqyow vzIA==
X-Received: by 10.181.28.3 with SMTP id jk3mr14570442wid.14.1416950845399; Tue, 25 Nov 2014 13:27:25 -0800 (PST)
MIME-Version: 1.0
Received: by 10.27.177.137 with HTTP; Tue, 25 Nov 2014 13:27:05 -0800 (PST)
In-Reply-To: <546FB4E7.7060704@nostrum.com>
References: <546FB4E7.7060704@nostrum.com>
From: Ira McDonald <blueroofmusic@gmail.com>
Date: Tue, 25 Nov 2014 16:27:05 -0500
Message-ID: <CAN40gSs3+tN80bm0hMPv5J=DVFnVkvJtNFS_rXrLzG=q-Gsacg@mail.gmail.com>
Subject: Re: Gen-art LC review: draft-mcdonald-ipps-uri-scheme-16
To: Robert Sparks <rjsparks@nostrum.com>, Ira McDonald <blueroofmusic@gmail.com>, Michael R Sweet <msweet@apple.com>
Content-Type: multipart/alternative; boundary="001a11c2a668cde8c60508b593c8"
Archived-At: http://mailarchive.ietf.org/arch/msg/ietf/OFlTLe5nKnY0C6TGTRw7nul7pHM
X-Mailman-Approved-At: Wed, 26 Nov 2014 08:05:14 -0800
Cc: "ietf@ietf.org" <ietf@ietf.org>, draft-mcdonald-ipps-uri-scheme.all@tools.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: Tue, 25 Nov 2014 21:27:31 -0000

Hi Robert,

Some replies and questions inline in BLUE.

Barry - there are several places below in Robert's and my comments
where we need your guidance, I think.

I'll try to turn out a new draft ASAP, but would appreciate Barry's and
Robert's guidance before posting it.

Cheers,
- Ira

Ira McDonald (Musician / Software Architect)
Co-Chair - TCG Trusted Mobility Solutions WG
Chair - Linux Foundation Open Printing WG
Secretary - IEEE-ISTO Printer Working Group
Co-Chair - IEEE-ISTO PWG Internet Printing Protocol WG
IETF Designated Expert - IPP & Printer MIB
Blue Roof Music / High North Inc
http://sites.google.com/site/blueroofmusic
http://sites.google.com/site/highnorthinc
mailto: blueroofmusic@gmail.com
Winter  579 Park Place  Saline, MI  48176  734-944-0094
Summer  PO Box 221  Grand Marais, MI 49839  906-494-2434


On Fri, Nov 21, 2014 at 4:55 PM, Robert Sparks <rjsparks@nostrum.com> wrote:

>  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>
> <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-mcdonald-ipps-uri-scheme-16
> Reviewer: Robert Sparks
> Review Date: 21-Nov-2014
> IETF LC End Date: 25-Nov-2014
> IESG Telechat date: 4-Dec-2014
>
> Summary: Almost ready, but has nits that should be addressed before
> publication as a Proposed Standard
>
> Nits/editorial comments:
>
> First: (For Barry as sponsoring AD and shepherd):
>
> I think you might want to say more about how this and the related PWG
> documents are being handled cross-organization.
>
> An RFC that normatively updates a document under some other organization's
> change control is an unusual thing. Usually there are parallel documents
> coordinating this. Is there such a parallel PWG doc this time?
>

<ira> (Slight clarification of Mike Sweet's earlier reply)
No.

Generally speaking, the PWG IPP WG doesn't develop parallel documents for
publication by both the PWG (IEEE-ISTO) and IETF - there is no point.  PWG
documents include IANA registration templates which go through the usual
registration process, independent of any IETF RFC publication.

However, since RFC 3510 is an IETF document that was produced by the (now
defunct) IETF IPP WG, and since this document is a parallel alternative to
RFC 3510 (and inherits most of the text of RFC 3510), we (the PWG IPP WG)
decided it should be submitted for publication by (and under the IP rules
of) the IETF rather than publishing it as a PWG standard.  And Barry has
graciously agreed to shepard the document...
</ira>

>
> Why aren't there RFC variants of the PWG docs (we've republished other
> organization's documents in the RFC series before...)
>

<ira> (Expanding my earlier reply)
The later versions of the IPP protocol (2.0, 2.1, and 2.2) defined in
IEEE-ISTO PWG
5100.12 normatively depend on over twenty other PWG specs (IPP and
otherwise),
so republishing them as IETF RFCs is not feasible.
</ira>

>
> Second: The 6 step construction in section 3 is a little odd. Why aren't
> steps 3-5 collapsed into one step that says "go do what https: says to do"?
> Split this way, especially with the repeated guidance in the security
> considerations section pointing somewhat loosely to 7320 and 5246 for
> things that "can be used to address this threat" looks like an opportunity
> for someone to get creative with how they check the certificate supplied by
> the server against the name in the URI. If you don't want anything but what
> happens in https to happen, I think it needs to be more clearly stated.
> Otherwise, doesn't this go off into RFC 6125 territory?
>

<ira>
We need some AD guidance from Barry here.  We could collapse steps 3-5 to a
pointer to HTTPS, if that's preferable.

We don't want anyone to "get creative" in checking a certificate, so
guidance in
how to say that best would be appreciated.
</ira>

>
> Lastly (and much smaller nits):
>
> There are several callouts from the text that look like references that
> are not represented in the references section.
> ID nits complains about all of these, and should make them easy to find
> and fix.
>

<ira>
I just ran ID Nits in the verbose mode and saved the output.

Unfortunately, the I-D submission checks didn't show any of these errors or
warnings.
We certainly need the extra boilerplate for the pre-RFC5378 work.  Several
of the
original authors of RFC 2910 and RFC 2911 are dead or unreachable AFAIK.

These ID Nits may take a little while to fix...
</ira>


> For example (from section 1.2):
>
>   2) Some existing IPP Client and IPP Printer implementations of HTTP
>       Upgrade [RFC 2717] do not perform upgrade at the beginning of
>
> <ira>
That's a typo - should have been [RFC2817].
</ira>

>  This reference is oddly constructed - please check early with the RFC
> Editor on whether they
> will take this, or want something a little different.
>
> [HTTP1.1]     HTTP/1.1.  See [RFC7230], [RFC7231], [RFC7232],
>               [RFC7233], [RFC7234], and [RFC7235].
>
> <ira>
We'll remove this composite reference - it's only ever used (incorrectly)
in the acronyms appendix.
</ira>

>  This line is wrong, and is causing idnits to complain once where it
> shouldn't.
> (The thing in the [] should be RFC7235, not 4):
>
>    [RFC7234]  Fielding, R., and J. Reschke, "Hypertext Transfer Protocol
>               (HTTP/1.1):  Authentication", RFC 7235, June 2014.
>
> <ira>
Yes - it's a cut-and-paste typo.
</ira>