Re: [Gen-art] Gen-ART Last Call review of draft-hethmon-mcmurray-ftpext-ftp-hosts-03

Jari Arkko <jari.arkko@piuha.net> Wed, 23 October 2013 17:41 UTC

Return-Path: <jari.arkko@piuha.net>
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 786FE11E8488 for <gen-art@ietfa.amsl.com>; Wed, 23 Oct 2013 10:41:06 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.584
X-Spam-Level:
X-Spam-Status: No, score=-102.584 tagged_above=-999 required=5 tests=[AWL=0.015, 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 w1aQt2TvXcto for <gen-art@ietfa.amsl.com>; Wed, 23 Oct 2013 10:41:02 -0700 (PDT)
Received: from p130.piuha.net (p130.piuha.net [193.234.218.130]) by ietfa.amsl.com (Postfix) with ESMTP id 63D7611E8463 for <gen-art@ietf.org>; Wed, 23 Oct 2013 10:40:19 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by p130.piuha.net (Postfix) with ESMTP id 7013E2CEB5; Wed, 23 Oct 2013 20:40:18 +0300 (EEST)
X-Virus-Scanned: amavisd-new at piuha.net
Received: from p130.piuha.net ([127.0.0.1]) by localhost (p130.piuha.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 7W-cMhCmEJsP; Wed, 23 Oct 2013 20:40:17 +0300 (EEST)
Received: from [127.0.0.1] (p130.piuha.net [IPv6:2a00:1d50:2::130]) by p130.piuha.net (Postfix) with ESMTP id F2B442CC48; Wed, 23 Oct 2013 20:40:13 +0300 (EEST)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 6.6 \(1510\))
From: Jari Arkko <jari.arkko@piuha.net>
In-Reply-To: <ABCAA4EF18F17B4FB619EA93DEF7939A2822B1E4@eusaamb107.ericsson.se>
Date: Thu, 24 Oct 2013 01:40:09 +0800
Content-Transfer-Encoding: quoted-printable
Message-Id: <663AB137-87A7-4E1C-97B2-36D5495BE550@piuha.net>
References: <ABCAA4EF18F17B4FB619EA93DEF7939A2822B1E4@eusaamb107.ericsson.se>
To: Meral Shirazipour <meral.shirazipour@ericsson.com>
X-Mailer: Apple Mail (2.1510)
Cc: "draft-hethmon-mcmurray-ftpext-ftp-hosts.all@tools.ietf.org" <draft-hethmon-mcmurray-ftpext-ftp-hosts.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-hethmon-mcmurray-ftpext-ftp-hosts-03
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: Wed, 23 Oct 2013 17:41:06 -0000

Thank you very much for this detailed review. Authors, do you have comments? While much of the comments are editorial, like Meral I had trouble understanding what the "chooses not conform" on page 17 means. Did you mean "chooses not to use"? Similarly, on page 18, I did not understand this: "a server implementation MUST treat an additional HOST command that was sent before a user has been authenticated as though a previous HOST command was not sent." Can you clarify?

Jari

On Oct 19, 2013, at 1:14 PM, Meral Shirazipour <meral.shirazipour@ericsson.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  .
>  
> Please resolve these comments along with any other Last Call comments you may receive.
>  
> Document: draft-hethmon-mcmurray-ftpext-ftp-hosts-03
> Reviewer: Meral Shirazipour
> Review Date: 2013-10-18
> IETF LC End Date: 2013-10-18
> IESG Telechat date: 2013-10-24
>  
>  
> Summary:
> This draft is ready to be published as Standard RFC but I do have some comments.
>  
>  
> Nits/editorial comments:
> Overall the draft was a bit hard to read but given the long time it has been around I presume it has passed implementers' test when it comes to clarity.
> Here are a few comments:
> -[Page 1], Asbtract: A sentence is missing between 1st and 2nd sentence: it should say what is the problem with the fact stated in the 1st sentence.
> -[Page 5], typo, "accomodate"-->"accommodate"  (two occurrences)
> -[Page 11] section 3.2.2. "the the server-PI "----> extra "the"
> -[Page 17], Section 3.3 "or otherwise chooses not conform" --> not clear what is not conform
> -[Page 18], Section 4, not clear what this sentence means:
> "a server implementation
>    MUST treat an additional HOST command that was sent before a user has
>    been authenticated as though a previous HOST command was not sent.
> "
>  
> Best Regards,
> Meral
>  
> ---
> Meral Shirazipour
> Ericsson
> Research
> www.ericsson.com
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art