Re: [mile] Reference and related nits in draft-ietf-mile-enum-reference-format

"Takeshi Takahashi" <takeshi_takahashi@nict.go.jp> Thu, 22 May 2014 11:44 UTC

Return-Path: <takeshi_takahashi@nict.go.jp>
X-Original-To: mile@ietfa.amsl.com
Delivered-To: mile@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D32F21A0195 for <mile@ietfa.amsl.com>; Thu, 22 May 2014 04:44:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.043
X-Spam-Level:
X-Spam-Status: No, score=-0.043 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_EQ_JP=1.244, HOST_EQ_JP=1.265, RP_MATCHES_RCVD=-0.651, SPF_PASS=-0.001] autolearn=no
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 9Vo7gZyxIn3R for <mile@ietfa.amsl.com>; Thu, 22 May 2014 04:44:57 -0700 (PDT)
Received: from ns2.nict.go.jp (ns2.nict.go.jp [IPv6:2001:df0:232:300::2]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id CBCA91A018A for <mile@ietf.org>; Thu, 22 May 2014 04:44:56 -0700 (PDT)
Received: from gw2.nict.go.jp (gw2 [133.243.18.251]) by ns2.nict.go.jp with ESMTP id s4MBilAn005880; Thu, 22 May 2014 20:44:47 +0900 (JST)
Received: from VAIO (ssh.nict.go.jp [133.243.3.49]) by gw2.nict.go.jp with ESMTP id s4MBik03021179; Thu, 22 May 2014 20:44:46 +0900 (JST)
From: Takeshi Takahashi <takeshi_takahashi@nict.go.jp>
To: 'Adam Montville' <adam@stoicsecurity.com>, "'Black, David'" <david.black@emc.com>
References: <8D3D17ACE214DC429325B2B98F3AE712076C55BC0D@MX15A.corp.emc.com> <4CD8DDF5-ECA1-4351-90DE-D8B213788A1C@stoicsecurity.com>
In-Reply-To: <4CD8DDF5-ECA1-4351-90DE-D8B213788A1C@stoicsecurity.com>
Date: Thu, 22 May 2014 20:44:45 +0900
Message-ID: <00a101cf75b3$3ae4e970$b0aebc50$@nict.go.jp>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQKc7BPik8IqmT4Sp0oJUR3hLlrCgAFkNJ1kmaZm/0A=
Content-Language: ja
X-Virus-Scanned: clamav-milter 0.97.8 at zenith2
X-Virus-Status: Clean
Archived-At: http://mailarchive.ietf.org/arch/msg/mile/ATVM8dywFZIjamhxFFev9cQv6Bo
Cc: mile@ietf.org, mile-chairs@tools.ietf.org
Subject: Re: [mile] Reference and related nits in draft-ietf-mile-enum-reference-format
X-BeenThere: mile@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Managed Incident Lightweight Exchange, IODEF extensions and RID exchanges" <mile.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mile>, <mailto:mile-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/mile/>
List-Post: <mailto:mile@ietf.org>
List-Help: <mailto:mile-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mile>, <mailto:mile-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 22 May 2014 11:44:59 -0000

Hi Adam,

I remember that we had discussion on the Index field last year.
We have a field for the Index and not for version number.
That is because "the Index field in the XML unambiguously indicates which
IANA registry entry is to be used to correctly reference the enumeration
specification, which avoids interpretation of version strings that may have
specification-specific formats" (copied from the draft).
Yes, this was what I understood in the discussion last year.

Now I am reading the document again, and have stuck on one question.
Maybe I have some misunderstanding or have forgotten some discussion, but I
would appreciate if you could clarify the issue below.

If we have Index field, do we need the Abbreviation field?

Below is the example described in the draft.
<EnumRef>
  <Abbreviation>CXI</Abbreviation>
  <Index>1</Index>
  <ID>CXI-1234-XYZ</ID>
</EnumRef>

But can we simply write like this?
<EnumRef>
  <Index>1</Index>
  <ID>CXI-1234-XYZ</ID>
</EnumRef>

If we have the Index value, we can uniquely identify the specification and
version, correct?

Take


> -----Original Message-----
> From: Adam Montville [mailto:adam@stoicsecurity.com]
> Sent: Monday, May 19, 2014 10:59 PM
> To: Black, David
> Cc: Alexey Melnikov; <mile@ietf.org>; mile-chairs@tools.ietf.org
> Subject: Re: Reference and related nits in
> draft-ietf-mile-enum-reference-format
> 
> David, thank you.  I've noted all of these and will make appropriate
> updates.
> 
> Are there other comments other than those posed by Alexey and David?
> 
> On May 18, 2014, at 7:39 PM, Black, David <david.black@emc.com> wrote:
> 
> > Hi Adam,
> >
> >> With respect to your first sentence above, I assert that the document
> >> does not "seem" to update ReferenceName, but actually does. :-)  Is
> >> it necessary to explicitly state the obvious in this case?  I'm happy
> >> to update the document, but I'm not sure I believe such a statement
> >> is necessary.  If it is necessary, then a single sentence can be added
> to the abstract making the statement.
> >> Thoughts here?
> >
> > In this case, it is necessary to state the obvious for the sake of
> > clarity - I've been there and had to do that ;-).  In addition to a
> > sentence in the abstract, the header on the title page needs an
"Updates:
> 5070 (if approved)"
> > line, and a section should be added entitled "Summary of Changes to RFC
> 5070"
> > that identifies the changes.
> >
> > For a somewhat longer recent example, see RFC 7146:
> > 	http://datatracker.ietf.org/doc/rfc7146/
> >
> > ----------------
> >
> > Here are some additional editorial nits that need attention ...
> >
> > a) I recently had another draft dinged by the IESG for using a
> > citation as a word in a sentence e.g., "[IODEF]" in the first sentence
> in Section 2:
> >
> >   The need is to place enumeration identifiers and their references in
> >   [IODEF]'s Reference class.
> >
> > Believe it or not, the appropriate change is "[IODEF]" -> "IODEF
> > [IODEF]" - I suggest just doing this and not asking why :-). The whole
> > draft should be checked for other uses of citations in this fashion.
> >
> > b) In addition, references should not be cited in the abstract, so I
> > believe the appropriate changes to the two citations in the abstract
are:
> >
> >   The Incident Object Description Exchange Format [IODEF] provides a
> >
> > 		"[IODEF]" -> "(IODEF)"
> >
> >   Reference class used to reference external entities (such as
> >   enumeration identifiers).  However, the method of external entity
> >   identification has been left unstructured.  This document describes
> a
> >   method to provide structure for referencing external entities for the
> >   [IODEF] Reference class.
> >
> > 		"[IODEF]" -> "IODEF"
> >
> > Otherwise, the draft still looks good.
> >
> > Thanks,
> > --David
> >
> >> -----Original Message-----
> >> From: mile [mailto:mile-bounces@ietf.org] On Behalf Of Adam Montville
> >> Sent: Sunday, May 18, 2014 7:58 PM
> >> To: Alexey Melnikov
> >> Cc: <mile@ietf.org>; mile-chairs@tools.ietf.org
> >> Subject: Re: [mile] Working Group Last Call for draft-ietf-mile-enum-
> >> reference-format
> >>
> >>
> >> On May 11, 2014, at 2:54 PM, Alexey Melnikov
> >> <alexey.melnikov@isode.com>
> >> wrote:
> >>
> >>>
> >>>> On 8 May 2014, at 12:30, "Takeshi Takahashi"
> >>>> <takeshi_takahashi@nict.go.jp>
> >> wrote:
> >>>>
> >>>> Greetings, all,
> >>>>
> >>>> This is the Working Group Last Call for
> >>>> draft-ietf-mile-enum-reference-format, IODEF Enumeration Reference
> Format.
> >>>>
> >>>> The last call starts now, and ends Monday 26 May 2014.
> >>>>
> >>>> Please read the latest revision of the draft,
> >>>>
> http://tools.ietf.org/html/draft-ietf-mile-enum-reference-format-04
> >>>> , and send your comments to the mailing list; we'd also appreciate
> >>>> a comment if you're fine with the draft as is.
> >>>
> >>> Hi,
> >>> Below is my review of the draft:
> >>>
> >>> General comment: Is this intended as Proposed Standard? The draft
> >>> header
> >> doesn't say the document type.
> >>
> >> I'll update this, thank you.
> >>
> >>>
> >>> The document seems to update ReferenceName syntax to allow for
> >>> nested XML
> >> elements. However the document doesn't say that.
> >>> In relationship to this: would this document cause any
> >>> interoperability
> >> issues with programs that can only parse the old schema?
> >>
> >> I am unsure about interoperability issues, but they should be minimal
> >> and are likely implementation specific.
> >>
> >> I tried an experiment where I generated some code (JAXB) for IODEF
> >> 1.0 and then fed it a modified recon example (Section 7.2 from RFC
> >> 5070).  The <ReferenceName> element contained an <EnumRef> element as
> >> opposed to the string "nmap".  Unmarshalling seemed to work without
> >> issue, but when examining the ReferenceName element , the only thing
> present was a newline character.
> >> In an example that doesn't use generated code but instead uses a SAX
> >> or DOM parser, we can grab the inner XML.  Using a validating
> >> DOM/SAX/other parser will, of course, break if the enumeration
> >> reference XML is embedded, so parsers will need to be updated.
> >>
> >> I'm not a professional implementor, so it would be good to hear
> >> considerations from others.  Then, should these considerations be
> >> placed in the document (I assume that was the good point of raising the
> issue)?
> >>
> >> With respect to your first sentence above, I assert that the document
> >> does not "seem" to update ReferenceName, but actually does. :-)  Is
> >> it necessary to explicitly state the obvious in this case?  I'm happy
> >> to update the document, but I'm not sure I believe such a statement
> >> is necessary.  If it is necessary, then a single sentence can be added
> to the abstract making the statement.
> >> Thoughts here?
> >>
> >>
> >>>
> >>>
> >>> 4  IANA Considerations
> >>>
> >>>     Note that certain name requests should not be permitted as either
> >>>     Full Name or Abbreviation entries for the requested IANA table.
> >>>
> >>> Do you mean restrictions imposed by the syntax described below or
> >>> something
> >> else? If you intended to exclude some specific keywords, where is the
> >> list of them defined? Either way, I think you need to clarify. If you
> >> just meant the ABNF restrictions, I suggest adding something like
> >> "The restrictions are specified in the ABNF below".
> >>>
> >>>     Allocation Policy: Expert Review [RFC5226] and Specification
> >>>     Required [RFC5226]
> >>>
> >>> Nit: I think it would be better to say "Specification Required
> >>> (which
> >> implies Expert Review)".
> >>>
> >>> _______________________________________________
> >>> mile mailing list
> >>> mile@ietf.org
> >>> https://www.ietf.org/mailman/listinfo/mile
> >