Re: [apps-discuss] draft-kucherawy-received-state

Mykyta Yevstifeyev <evnikita2@gmail.com> Tue, 10 January 2012 07:33 UTC

Return-Path: <evnikita2@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 E5D1221F8507 for <apps-discuss@ietfa.amsl.com>; Mon, 9 Jan 2012 23:33:37 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.544
X-Spam-Level:
X-Spam-Status: No, score=-3.544 tagged_above=-999 required=5 tests=[AWL=0.055, BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1]
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 wHJl6mmxUDpm for <apps-discuss@ietfa.amsl.com>; Mon, 9 Jan 2012 23:33:37 -0800 (PST)
Received: from mail-tul01m020-f172.google.com (mail-tul01m020-f172.google.com [209.85.214.172]) by ietfa.amsl.com (Postfix) with ESMTP id 6404821F8504 for <apps-discuss@ietf.org>; Mon, 9 Jan 2012 23:33:37 -0800 (PST)
Received: by obcuz6 with SMTP id uz6so5734871obc.31 for <apps-discuss@ietf.org>; Mon, 09 Jan 2012 23:33:37 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=Xl39nGVsDBCUJ/SjOamOXE1f7frpYk0+NnyO67vNqUc=; b=v+VjRWjjS+3sglECP4gWFFHdUnWGaIYlYJFu5omExw2e+fro9GofA8xdMKYf1Vxr2Y GXAhMx0p6NHY3e5Wn97f+0OvevFQpVKvUC8koTnxYYH8YvmzfvcOW+Pbir6IFCp5VERD QpeDbaDU5DO7TrTiJkbtGrWNEv4vK9gXr+7b0=
MIME-Version: 1.0
Received: by 10.182.47.100 with SMTP id c4mr17674798obn.54.1326180817034; Mon, 09 Jan 2012 23:33:37 -0800 (PST)
Received: by 10.182.213.71 with HTTP; Mon, 9 Jan 2012 23:33:36 -0800 (PST)
In-Reply-To: <F5833273385BB34F99288B3648C4F06F19C6C157CC@EXCH-C2.corp.cloudmark.com>
References: <F5833273385BB34F99288B3648C4F06F19C6C157BD@EXCH-C2.corp.cloudmark.com> <CADBvc9_Re+FqjAF7GC42N319m-4dyAGwnD0G0yhbFKCRwd4dLA@mail.gmail.com> <F5833273385BB34F99288B3648C4F06F19C6C157CC@EXCH-C2.corp.cloudmark.com>
Date: Tue, 10 Jan 2012 09:33:36 +0200
Message-ID: <CADBvc98DKENYA0JKnz7f90RQvbdopnauGZV_R24Lb_k5waOAcA@mail.gmail.com>
From: Mykyta Yevstifeyev <evnikita2@gmail.com>
To: "Murray S. Kucherawy" <msk@cloudmark.com>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable
Cc: "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Subject: Re: [apps-discuss] draft-kucherawy-received-state
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, 10 Jan 2012 07:33:38 -0000

2012/1/10 Murray S. Kucherawy <msk@cloudmark.com>:
>> -----Original Message-----
>> From: Mykyta Yevstifeyev [mailto:evnikita2@gmail.com]
>> Sent: Monday, January 09, 2012 11:03 PM
>> To: Murray S. Kucherawy
>> Cc: apps-discuss@ietf.org
>> Subject: Re: [apps-discuss] draft-kucherawy-received-state
>>
>> Hi Murray,
>
> Hello Mykyta,
>
>> This seems to be an interesting document (as for me).  You only have
>> made two mistakes in ABNF block:
>>
>>       State = "state" queue-state-keyword
>>
>> should be
>>
>>       State = "state" FWS queue-state-keyword
>>                 ; <FWS> defined in [RFC5322]
>>
>> Additionally, this:
>>
>>    A transfer agent making use of this extension MAY also include header
>>    field comments to provide additional information.
>>
>> doesn't suit ABNF as well.  RFC 5321 defined <Stamp> as follows:
>
> Actually "state" is covered by this, from RFC5322:
>
>        Additional-Registered-Clauses  = CFWS Atom FWS String
>
> We're defining a new Atom and String in this context.  So really we want:
>
>        State = CFWS "state" FWS queue-state-keyword
>            ; FWS and CFWS are defined in [MAIL]
>
> ...which matches the trace information clauses defined in RFC5322.  I've updated it to match this.

Yes, this is better.  And with respect to allowing comments after
separate 'state' clauses?

Mykyta

>
> Thanks,
> -MSK
> _______________________________________________
> apps-discuss mailing list
> apps-discuss@ietf.org
> https://www.ietf.org/mailman/listinfo/apps-discuss