Re: [urn] Expert review needed

worley@ariadne.com (Dale R. Worley) Sat, 20 January 2018 03:24 UTC

Return-Path: <worley@alum.mit.edu>
X-Original-To: urn@ietfa.amsl.com
Delivered-To: urn@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DC754126CC4 for <urn@ietfa.amsl.com>; Fri, 19 Jan 2018 19:24:46 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.685
X-Spam-Level:
X-Spam-Status: No, score=-1.685 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HEADER_FROM_DIFFERENT_DOMAINS=0.25, RCVD_IN_DNSWL_LOW=-0.7, SPF_SOFTFAIL=0.665] autolearn=no autolearn_force=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 MPJs-9161Ot5 for <urn@ietfa.amsl.com>; Fri, 19 Jan 2018 19:24:44 -0800 (PST)
Received: from resqmta-ch2-11v.sys.comcast.net (resqmta-ch2-11v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:43]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C2F16126CBF for <urn@ietf.org>; Fri, 19 Jan 2018 19:24:44 -0800 (PST)
Received: from resomta-ch2-14v.sys.comcast.net ([69.252.207.110]) by resqmta-ch2-11v.sys.comcast.net with ESMTP id cjliev7NLherIcjlreWyoW; Sat, 20 Jan 2018 03:24:43 +0000
Received: from hobgoblin.ariadne.com ([IPv6:2601:192:4603:9471:222:fbff:fe91:d396]) by resomta-ch2-14v.sys.comcast.net with SMTP id cjlqerWPi9aG0cjlrehSCo; Sat, 20 Jan 2018 03:24:43 +0000
Received: from hobgoblin.ariadne.com (hobgoblin.ariadne.com [127.0.0.1]) by hobgoblin.ariadne.com (8.14.7/8.14.7) with ESMTP id w0K3OfDH014043; Fri, 19 Jan 2018 22:24:41 -0500
Received: (from worley@localhost) by hobgoblin.ariadne.com (8.14.7/8.14.7/Submit) id w0K3Oe88014040; Fri, 19 Jan 2018 22:24:40 -0500
X-Authentication-Warning: hobgoblin.ariadne.com: worley set sender to worley@alum.mit.edu using -f
From: worley@ariadne.com
To: urn@ietf.org, r_atarius@yahoo.com
In-Reply-To: <0456A85D-1D83-420B-AA2E-DCD300408FF2@nostrum.com> (ben@nostrum.com)
Sender: worley@ariadne.com
Date: Fri, 19 Jan 2018 22:24:40 -0500
Message-ID: <87fu716wlj.fsf@hobgoblin.ariadne.com>
X-CMAE-Envelope: MS4wfOmYIBc0CyDHvILlEwveeCioHQYJzSxW+3ohoFUK8dSJgIGS16iunsfct/qL1VGjtIWT2zm0uR+OIOqVxm800UFAlc5rsrk5p3tQ5bYX6rXdpzgS+9WE Un9Liz3ByEvv1gtqGF4I8hXYCnJgfWxbcv/3q/8BGwuG/I7L4jpn9ddkSd8+ygyD5dEBRqN7jYFCtQ==
Archived-At: <https://mailarchive.ietf.org/arch/msg/urn/Ngldk2Ejgwx9Klq1PAUpnniWGrY>
Subject: Re: [urn] Expert review needed
X-BeenThere: urn@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Revisions to URN RFCs <urn.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/urn>, <mailto:urn-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/urn/>
List-Post: <mailto:urn@ietf.org>
List-Help: <mailto:urn-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/urn>, <mailto:urn-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 20 Jan 2018 03:24:47 -0000

Comments on draft-atarius-dispatch-meid-urn-14:

Summary:  This draft is useful and sensible and supplies a known need.
However, it needs a careful editing to get it cleaned up for
publication.

Possible major or global items:

As far as I can tell, the MEID check digit is "not transmitted" in the
sense that it's not part of the URN itself.  You should tidy up the
language used in this document.  In particular, you have to decide
whether the MEID contains or does not contain the check digit, and get
all the wording aligned.  E.g., the MEID is 15 hex digits, of which
only the first 14 are used in the URN (the check digit not being
transmitted), OR the MEID is 14 hex digits, of which all are used in
the URN, and the check digit is not transmitted as it isn't part of
the MEID.

If you allow that future changes will only be done via RFC, then you
can omit all mention of future-pp2-specifier in this RFC, as they will
be defined in later RFCs.  That could allow you to simplify this RFC
considerably (especially the ABNF), as it would only have to deal with
the one identifier class ("meid") that has already been defined.

Minor items:

Abstract

   The structure
   of an MEID is 15 hexadecimal encoded digits long and ...

s/hexadecimal encoded digits/hexadecimal digits/

1.  Introduction

   Document [RFC7254] defines a URN Namespace and an NSS for the IMEI.

I think the usual usage is to allow an RFC reference to be a noun, so
"Document" is redundant.

   For cases where the mobile equipment uses decimal values (i.e., IMEI)
   as an identity ...

The phrase "decimal values (i.e., IMEI)" is the wrong way around --
the important thing is that the identifier is an IMEI, it's just a
consequence that that the identifier is composed of decimal digits.
So s/decimal values (i.e., IMEI)/IMEI/.

   Whilst
   this specification currently specifies only the MEID NSS under the
   '3gpp2' NID, additional NSS under the '3gpp2' NID may be specified in
   the future by the 3GPP2.

This isn't particularly clear at this point in the document.  A fuller
explanation would be clearer as part of the introduction:

   This specification defines only NSSs constructed from MEIDs under
   the '3gpp2' NID.  These NSSs start with "meid:" in order to
   identify them as such.  Additional types of NSS under the '3gpp2'
   NID may be specified in the future by the 3GPP2 via additional
   specifications.

--

   The hexadecimal MEID is 15 hexadecimal digits long ...

s/The hexadecimal MEID/The MEID/

3.  Namespace Registration Template

   pp2-urn-char         = ALPHA / DIGIT / "-" / "." / "_" / "%" / ":"

% is not allowed as an independent character.  Looking at the
definition of URN (RFC 8141), an NSS is composed of pchar's and "/".
But "%" can only be a pchar as a percent-encoding (pct-encoded).

You probably want

   pp2-urn-char         = ALPHA / DIGIT / "-" / "." / "_" / pct-encoded / ":"

You may want to allow the value part of a future-param to be empty:

   future-param         = par-name [ EQUAL [ par-value ] ]

--

   <future-pp2-specifier> and <pp2-defined-nonempty> can comprise any
   ASCII characters compliant with URN syntax in [RFC5234].

This is not correct, as the URN syntax allows any pchar, whereas the
pp2-defined-nonempty does not allow the sub-delims "!", "$", "&", "'",
"(", ")", "*", "+", and ",", which are all valid pchars.

   Identifiers in the '3gpp2' NID are defined and assigned by the 3GPP2
   or an agency appointed by 3GPP2 after ensuring that the URNs to be
   assigned are unique.  Uniqueness is achieved by checking against the
   IANA registry of previously assigned names.

The last sentence of this paragraph makes no sense.  It seems to me
that it could be safely omitted.  OTOH, replacing "IANA" with the name
of the correct organization may also work.

   As the NID sought is '3gpp2', and 3GPP2 is the long standing acronym
   for the standards organization which includes the mobile phone
   operators, the URN should also persist indefinitely (at least as long
   as there is a need for its use).  The assignment process guarantees
   that names are not reassigned.  The binding between the name and its
   resource is permanent.

The phrase "3GPP2 is the long standing acronym for the standards
organization which includes the mobile phone operators" doesn't seem
to be useful -- the connection between the NID "3gpp2" and the 3GPP2
organization is assured to be permanent due to IANA procedures.  The
important part of the paragraph is that 3GPP2 has procedures to ensure
assignments within the NID are unique and persistent.

   The 3GPP2 or its approved agency will manage the <NSS> (including
   '3gpp2') and <future-pp2-specifier> identifier resources to maintain
   uniqueness.  The process for MEID assignment is documented in
   [SC.R4002-0].

I think you mean "'meid'" instead of "'3gpp2'" -- 3GPP2 does not
manage the "3gpp2" part of the URN itself.

   Two 3GPP2 MEID URNs are equivalent if they have the same 'meidval'
   and the same parameter values in the same sequential order.  All of
   these comparisons are to be case-insensitive.

This rule does not specify how future-pp2-specifier's are to be
compared.

   Any identifier in '3gpp2' NSS can be compared using the normal
   mechanisms for percent-encoded UTF-8 strings (see [RFC3629]).

RFC 3629 only describes the UTF-8 encoding itself, not how
percent-encoding of it is to be handled.  Perhaps you meant a
different RFC?

4.1.  MEID Parameters

   Any future change to the format of the 'meid' NSS requires the use of
   the procedure for URN NSS changes (currently through the publication
   of a future Informational RFCs approved by IETF consensus).

You have a choice here.  If you allow that future changes will only be
done via RFC, then you can omit all mention of future-pp2-specifier in
this RFC, as they will be defined in later RFCs.  That could allow you
to simplify this RFC considerably, as it would only have to deal with
identifier classes that have already been defined.

4.2.4.  Hexadecimal Encoding

This section is interesting but irrelevant.  As it says, "the
actual storage format in memory is implementation specific".  As such,
the only storage format that is relevant to this document is how the
abstract MEID (a sequence of 14 or 15 elements drawn from the set of
hex digits) is represented in the URN.

8.  Security and privacy considerations

I haven't checked, but I assume that you've constructed this section
to closely parallel section 8 of RFC 7254 (the IMEI URN).  That will
ensure that privacy concerns are satisfied.  (RFC 7253 was delayed
considerably due to privacy concerns.)

10.1.  Normative References

   [draft-atarius-dispatch-meid-urn-as-instanceid]
              Atarius, R., "S.R0048-A: Using the Mobile Equipment
              Identity (MEID) Uniform Resource Name (URN) as an Instance
              ID", Internet Draft draft-atarius-dispatch-meid-urn-as-
              instanceid, January 2018.

This isn't a normative reference -- this draft does not normatively
depend on "MEID as Instance ID".  (Contrarily, that draft *does*
normatively depend on this draft.)

Also, its title does not start with "S.R0048-A:".

Dale