Re: Last Call: <draft-melnikov-mmhs-header-fields-04.txt> (Registration of Military Message Handling System (MMHS) header fields for use in Internet Mail) to Informational RFC

Mykyta Yevstifeyev <evnikita2@gmail.com> Thu, 15 September 2011 16:44 UTC

Return-Path: <evnikita2@gmail.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C99A121F8BD3 for <ietf@ietfa.amsl.com>; Thu, 15 Sep 2011 09:44:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.487
X-Spam-Level:
X-Spam-Status: No, score=-3.487 tagged_above=-999 required=5 tests=[AWL=0.112, 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 T3Kkd8Sf6Kyo for <ietf@ietfa.amsl.com>; Thu, 15 Sep 2011 09:44:07 -0700 (PDT)
Received: from mail-ey0-f172.google.com (mail-ey0-f172.google.com [209.85.215.172]) by ietfa.amsl.com (Postfix) with ESMTP id 0954F21F8BCB for <ietf@ietf.org>; Thu, 15 Sep 2011 09:44:06 -0700 (PDT)
Received: by eye4 with SMTP id 4so1676696eye.31 for <ietf@ietf.org>; Thu, 15 Sep 2011 09:46:18 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:content-type:content-transfer-encoding; bh=zdGm93VRV8bZTs5TfT6YmMjfPxuj/hPPnT6pcj2dyxQ=; b=uW3voWqPwtXjQ738qBZEXiBHZ8r+ABfEU+1Jh+NMc/s3tYS/I478Ivhex7fQACa/KB EX0puieoyPuQAojX8dTTsLtHL/u0e6qKCJ8KY2hu23CxNLbpEx7kAa6Bp7khVSv7YHh9 UnWsiiCPJHcx1exQ82ijyIR3TgQ44Y1v9LsPc=
Received: by 10.223.28.10 with SMTP id k10mr1089072fac.128.1316105178567; Thu, 15 Sep 2011 09:46:18 -0700 (PDT)
Received: from [127.0.0.1] ([195.191.104.224]) by mx.google.com with ESMTPS id w6sm7580014fah.0.2011.09.15.09.46.16 (version=SSLv3 cipher=OTHER); Thu, 15 Sep 2011 09:46:17 -0700 (PDT)
Message-ID: <4E722BF9.2010502@gmail.com>
Date: Thu, 15 Sep 2011 19:46:49 +0300
From: Mykyta Yevstifeyev <evnikita2@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:6.0.2) Gecko/20110902 Thunderbird/6.0.2
MIME-Version: 1.0
To: IETF Discussion <ietf@ietf.org>, draft-melnikov-mmhs-header-fields@tools.ietf.org
Subject: Re: Last Call: <draft-melnikov-mmhs-header-fields-04.txt> (Registration of Military Message Handling System (MMHS) header fields for use in Internet Mail) to Informational RFC
References: <20110914195304.6235.35027.idtracker@ietfa.amsl.com>
In-Reply-To: <20110914195304.6235.35027.idtracker@ietfa.amsl.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
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: Thu, 15 Sep 2011 16:44:07 -0000

So some small comments:

1) A nit in Change controller field for all the header fields: you 
should either change it to IESG (iesg@ietf.org) or IETF with subsequent 
address ietf@ietf.org.

2) Also, it might be useful to point out in the sub-sections of Section 
3 which ABNF productions do the header fields match.

3) Section 4:

>   Terms not defined here are taken from [RFC5322] and [RFC2156].

You probably missed RFC 5234 (with e. g. <DQUOTE>) here.

Some ABNF nits:

4)

>      military-string = 1*69( ps-char )

No harm to change it to "1*69ps-char"

5)

>      std-precedence = "deferred" / "routine" / "priority" /
>                       "immediate" / "flash" / "override"
>      std-message-type = "exercise" / "operation" / "project" /  "drill"

Is there an ability to extend the allowed range of parameters here?

6)

>      nonneg-integer = "0" / (NZ-DIGIT *DIGIT)

Should this be

>      nonneg-integer = "0" / ( [ "+" ] NZ-DIGIT *DIGIT)

as non-negative integer implies that it may be prepended by "+" (as well 
as zero - I don't know exactly).

7)

>      military-string-sequence = military-string
>                          [ [FWS] ";" [FWS] military-string-sequence ]

I think

>      military-string-sequence = military-string
>                          [ [FWS] ";" [FWS] military-string ]

is just the same but improves readability (unless you make this 
intentionally, but I see no reason).

8)

>      Subject-Indicator-Codes = "MMHS-Subject-Indicator-Codes:"
>                                [FWS] [sic-sequence] [FWS] CRLF

Can the header field appear to be:

>      MMHS-Subject-Indicator-Codes:

as you allow.  Maybe removing [] around sic-sequence?

9) I'd like you hereby disallowed further registration of header fields 
beginning with "MMHS", likewise RFC 5504 "Downgraded" prefix 
(http://tools.ietf.org/html/rfc5504#page-18).

Mykyta Yevstifeyev

14.09.2011 22:53, The IESG wrote:
> The IESG has received a request from an individual submitter to consider
> the following document:
> - 'Registration of Military Message Handling System (MMHS) header fields
>     for use in Internet Mail'
>    <draft-melnikov-mmhs-header-fields-04.txt>  as an Informational RFC