[radext] [IANA #922685] expert review for draft-ietf-radext-ip-port-radius-ext (IP Flow Information Export (IPFIX) Entities)

"Amanda Baber via RT" <drafts-expert-review-comment@iana.org> Sat, 20 August 2016 00:29 UTC

Return-Path: <iana-shared@icann.org>
X-Original-To: expand-draft-ietf-radext-ip-port-radius-ext.all@virtual.ietf.org
Delivered-To: radext@ietfa.amsl.com
Received: by ietfa.amsl.com (Postfix, from userid 65534) id 12FF912B074; Fri, 19 Aug 2016 17:29:41 -0700 (PDT)
X-Original-To: xfilter-draft-ietf-radext-ip-port-radius-ext.all@ietfa.amsl.com
Delivered-To: xfilter-draft-ietf-radext-ip-port-radius-ext.all@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7F91B12D0CF for <xfilter-draft-ietf-radext-ip-port-radius-ext.all@ietfa.amsl.com>; Fri, 19 Aug 2016 17:29:41 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.427
X-Spam-Level:
X-Spam-Status: No, score=-4.427 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HEADER_FROM_DIFFERENT_DOMAINS=0.001, MISSING_HEADERS=1.021, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H2=-0.001, RP_MATCHES_RCVD=-1.247, SPF_PASS=-0.001] autolearn=ham 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 vbjgadnEiRfs for <xfilter-draft-ietf-radext-ip-port-radius-ext.all@ietfa.amsl.com>; Fri, 19 Aug 2016 17:29:40 -0700 (PDT)
Received: from smtp02.icann.org (smtp01.icann.org [192.0.46.81]) (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 3061E12B074 for <draft-ietf-radext-ip-port-radius-ext.all@ietf.org>; Fri, 19 Aug 2016 17:29:40 -0700 (PDT)
Received: from request3.lax.icann.org (request1.lax.icann.org [10.32.11.221]) by smtp02.icann.org (Postfix) with ESMTP id 25BC0E3263 for <draft-ietf-radext-ip-port-radius-ext.all@ietf.org>; Sat, 20 Aug 2016 00:29:39 +0000 (UTC)
Received: by request3.lax.icann.org (Postfix, from userid 48) id EB464C205C1; Sat, 20 Aug 2016 00:29:38 +0000 (UTC)
RT-Owner: sabrina.tanamal
From: Amanda Baber via RT <drafts-expert-review-comment@iana.org>
In-Reply-To: <rt-4.2.9-29488-1470865629-844.922685-9-0@icann.org>
References: <RT-Ticket-922685@icann.org> <rt-4.2.9-29488-1470865629-844.922685-9-0@icann.org>
Message-ID: <rt-4.2.9-19936-1471652978-1944.922685-9-0@icann.org>
X-RT-Loop-Prevention: IANA
X-RT-Ticket: IANA #922685
X-Managed-BY: RT 4.2.9 (http://www.bestpractical.com/rt/)
X-RT-Originator: amanda.baber@icann.org
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
Precedence: bulk
Date: Sat, 20 Aug 2016 00:29:38 +0000
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Resent-From: alias-bounces@ietf.org
Resent-To: dean.cheng@huawei.com, jouni.nospam@gmail.com, mohamed.boucadair@orange.com, ssenthil@cisco.com, stefan.winter@restena.lu, lionel.morand@orange.com, bclaise@cisco.com, joelja@bogus.com, Kathleen.Moriarty.ietf@gmail.com, radext@ietf.org
Resent-Message-Id: <20160820002942.12FF912B074@ietfa.amsl.com>
Resent-Date: Fri, 19 Aug 2016 17:29:41 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/radext/rpqYhgQu_eBnqTfeuOMejRR1pLE>
Cc: draft-ietf-radext-ip-port-radius-ext.all@ietf.org
Subject: [radext] [IANA #922685] expert review for draft-ietf-radext-ip-port-radius-ext (IP Flow Information Export (IPFIX) Entities)
X-BeenThere: radext@ietf.org
X-Mailman-Version: 2.1.17
Reply-To: drafts-expert-review-comment@iana.org
List-Id: RADIUS EXTensions working group discussion list <radext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/radext>, <mailto:radext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/radext/>
List-Post: <mailto:radext@ietf.org>
List-Help: <mailto:radext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/radext>, <mailto:radext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 20 Aug 2016 00:29:42 -0000

Dear Authors,

The experts for the IPFIX IE registry have returned the following review:

In general, the Information Elements in draft-ietf-radext-ip-port-radius-ext are so underspecified as to be unimplementable. They should not be added to the registry in their present form. The authors are advised to read RFC 7013, especially Section 4, which provides useful information on defining Information Elements. Specifically:

The Information Element transportType is underspecified: (a) I presume this is in reference to sourceTransportPort and destinationTransportPort, but the description must say this if it is the case; (b) It's not clear at all from the description in what context this distinction is useful; (c) What's an ICMP identifier?

In addition, the description of transportType appears to create a table which should probably be handled as a subregistry. See See RFC7013 section 4.7. for advice on the creation of tables without subregistries (in short, "don't".)

The Information Element natTransportLimit has an inappropriate name; it does not describe that which it (presumably) is supposed to represent (see RFC 7013 section 4.1). In addition, it is underspecified. It is impossible to implement from the description. Is the field IPv4 specific, or is IPv6 supported as well? (If not, why not?)

The Information Element localID has an inappropriate name; it is far too general (see RFC 7013 section 4.1). It uses an inappropriate abstract data type (addresses should never be represented as UTF-8 strings in IPFIX, see RFC 7013 section 4.2). It is underspecified as well as poorly designed. Without the ability to disambiguate the type of information in the field, this is not a useful Information Element. Without a complete enumeration of possible types (n.b. 'etc.' in the description), it is not a useful Information Element. Its purpose is unclear from its description; further, it appears to violate the following guidance in RFC 7013 section 4: "The Information Element must be unique within the registry, and its description must represent a substantially different meaning from that of any existing Information Element. An existing Information Element that can be reused for a given purpose should be reused."

Best regards,

Amanda Baber
IANA Lead Specialist
ICANN