Re: [apps-discuss] APPSDIR review of draft-melnikov-smtp-priority-tunneling-02

Alexey Melnikov <alexey.melnikov@isode.com> Thu, 12 July 2012 10:09 UTC

Return-Path: <alexey.melnikov@isode.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 8042B21F87B8 for <apps-discuss@ietfa.amsl.com>; Thu, 12 Jul 2012 03:09:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.986
X-Spam-Level:
X-Spam-Status: No, score=-102.986 tagged_above=-999 required=5 tests=[AWL=-0.387, 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 eVJTdtyNal0Z for <apps-discuss@ietfa.amsl.com>; Thu, 12 Jul 2012 03:09:09 -0700 (PDT)
Received: from statler.isode.com (statler.isode.com [62.3.217.254]) by ietfa.amsl.com (Postfix) with ESMTP id A3BF821F87B7 for <apps-discuss@ietf.org>; Thu, 12 Jul 2012 03:09:09 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1342087781; d=isode.com; s=selector; i=@isode.com; bh=Nev2CyRu95Kau3bAN3/WTcHrOGRAGOOtxkdyNTx+zMI=; 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=ZadmGVLt98LeBgrhWC2/3OyziTN0o+L5msfaQeOGP9G82UcgqRMjBkpSBH9mz/CKsLl8Q2 Yy9iwJzwZ8QnI2h6NEQaQ5Wliy9QTecvruxbP/Ad2maxWk+prWdDeBvvzLMrf0UqC/r5/B ejNJKtGlyfPd25l0uMYIe3MRLhIsPsQ=;
Received: from [192.168.1.144] ((unknown) [62.3.217.253]) by statler.isode.com (submission channel) via TCP with ESMTPSA id <T=6iZABcm4xk@statler.isode.com>; Thu, 12 Jul 2012 11:09:41 +0100
X-SMTP-Protocol-Errors: NORDNS PIPELINING
Message-ID: <4FFEA279.5020909@isode.com>
Date: Thu, 12 Jul 2012 11:10:01 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20120614 Thunderbird/13.0.1
To: Joseph Yee <jyee@afilias.info>
References: <CAF1dMVGhOaox+k7AFt+UOJOBWAsJfWAhYH7UOZTeiyVyuwzuvw@mail.gmail.com>
In-Reply-To: <CAF1dMVGhOaox+k7AFt+UOJOBWAsJfWAhYH7UOZTeiyVyuwzuvw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: apps-discuss@ietf.org, draft-melnikov-smtp-priority-tunneling.all@tools.ietf.org
Subject: Re: [apps-discuss] APPSDIR review of draft-melnikov-smtp-priority-tunneling-02
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: Thu, 12 Jul 2012 10:09:10 -0000

Hi Joseph,
Thank you for the review and sorry for the late response.

On 22/06/2012 22:48, Joseph Yee wrote:
> I have been selected as the Applications Area Directorate reviewer for
> this draft (for background on appsdir, please see
> http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate
> ).
>
> 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-melnikov-smtp-priority-tunneling-02
> Title: Tunneling of Message Transfer Priorities
> Reviewer: Joseph Yee
> Review Date: 2012-06-22
>
> Summary:
>
>      This draft is ready for publication as an Experimental RFC.
>
> Major Issues:
>
>      none
>
> Minor Issues:
>
>      none
>
> Nits:
>
>      (1)
>          At section 1 Introduction, the text in third paragraph talks
> about unextended MUA in scenario 2, but in the detailed description at
> 5th paragraph, it says "The client is extended via a plugin.....".
> Although the text meant many MUA plugin do not cover SMTP protocol
> features, the word "extended" in detailed description has some
> confusion.  Suggest to change it to "The client is extendable via
> ...."

Thanks, changed.

>      (2)
>          Section 1 Introduction, 3rd paragraph, typo
>          s/scenarious/scenarios/

Fixed, thanks.

>      (3)
>          Several references (Normative References) were not used.
> Although some textes "touched" keywords like 'MSA', it did not cite
> specific rules or concepts.  So either add the reference back in text
> or remove the references work, IMHO.
>          - [RFC2034]
>          - [RFC5321]
>          - [RFC5248]
>          - [RFC6409]

Good point. I will eliminate some of these and will reference the others.