Re: [Dime] [apps-discuss] Review of: draft-ietf-dime-rfc4005bis-09

Glen Zorn <glenzorn@gmail.com> Sun, 07 October 2012 06:48 UTC

Return-Path: <glenzorn@gmail.com>
X-Original-To: dime@ietfa.amsl.com
Delivered-To: dime@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 480A721F8472; Sat, 6 Oct 2012 23:48:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.299
X-Spam-Level:
X-Spam-Status: No, score=-4.299 tagged_above=-999 required=5 tests=[AWL=-1.300, BAYES_00=-2.599, J_CHICKENPOX_51=0.6, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 9NEgDXuHfMMt; Sat, 6 Oct 2012 23:48:58 -0700 (PDT)
Received: from mail-pb0-f44.google.com (mail-pb0-f44.google.com [209.85.160.44]) by ietfa.amsl.com (Postfix) with ESMTP id 3CFF121F844F; Sat, 6 Oct 2012 23:48:58 -0700 (PDT)
Received: by mail-pb0-f44.google.com with SMTP id ro8so3190266pbb.31 for <multiple recipients>; Sat, 06 Oct 2012 23:48:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=C9SNFUuwPj5ePLgyvbx0YQpxcdo+mXJPaBv8vXcdLY4=; b=nvdt4E/YfTrBUkYxmgCLszGiTtpGAa+kesEFALNW/zf2eKjW7ff4ud/VDlJaiicPVT EamNvk4a0Iup/DSLCHfzYbA9frK0+T0w4j9N1ReLVx7/44r7GJm+XqyU1AJsNhSaGU51 F62WqiuIW0nxmgAhV2EHH6zoD2uWXQbTORV+VHsz1Zlz5tMDOkt+jV5aEyt1mQ3+EiMn dUe3nUeGL/AXqZGEqc75a60fThwgYY8Bo4zhqz0ow7gYPXyJCl+Oy/fg0sR+CIBTTW38 vrJGbN3HlDw2Lb1B6LzA9GWMNtb+YzD0S5RyG8yeb7oChZQX9cnm9kKaoLjzNr4KM4aX e7ew==
Received: by 10.68.242.9 with SMTP id wm9mr43292570pbc.62.1349592537886; Sat, 06 Oct 2012 23:48:57 -0700 (PDT)
Received: from [192.168.0.102] (ppp-115-87-90-243.revip4.asianet.co.th. [115.87.90.243]) by mx.google.com with ESMTPS id nd6sm5168011pbc.68.2012.10.06.23.48.54 (version=SSLv3 cipher=OTHER); Sat, 06 Oct 2012 23:48:57 -0700 (PDT)
Message-ID: <507125D5.4050304@gmail.com>
Date: Sun, 07 Oct 2012 13:48:53 +0700
From: Glen Zorn <glenzorn@gmail.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120914 Thunderbird/15.0.1
MIME-Version: 1.0
To: dcrocker@bbiw.net
References: <50161436.8080900@bbiw.net> <50694C10.4050508@gmail.com> <5069C58D.1090200@dcrocker.net>
In-Reply-To: <5069C58D.1090200@dcrocker.net>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: Dave Crocker <dhc@dcrocker.net>, draft-ietf-dime-rfc4005bis.all@tools.ietf.org, apps-discuss@ietf.org, dime mailing list <dime@ietf.org>, iesg <iesg@ietf.org>
Subject: Re: [Dime] [apps-discuss] Review of: draft-ietf-dime-rfc4005bis-09
X-BeenThere: dime@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Diameter Maintanence and Extentions Working Group <dime.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dime>, <mailto:dime-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dime>
List-Post: <mailto:dime@ietf.org>
List-Help: <mailto:dime-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dime>, <mailto:dime-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 07 Oct 2012 06:48:59 -0000

On 10/01/2012 11:32 PM, Dave Crocker wrote:

> Glen, et al,
 >
 >
 > On 10/1/2012 12:53 AM, Glen Zorn wrote:
 >> On 07/30/2012 11:57 AM, Dave Crocker wrote:
 >>>> 1. Introduction
 >>>>
 >>>> This document describes the Diameter protocol application used
 >>>> for AAA in the Network Access Server (NAS) environment. When
 >>>> combined with the Diameter Base protocol
 >>>> [I-D.ietf-dime-rfc3588bis], Transport Profile [RFC3539], and
 >>>> EAP [RFC4072] specifications, this specification satisfies the
 >>>> NAS-related requirements defined in Aboba, et al. [RFC2989] and
 >>>> Beadles & Mitton [RFC3169].
 >>>
 >>> This assertion raises a dilemma: The cited documents are
 >>> extensive and the topic(s?) complex. There is no way to know what
 >>> details are being referred to, to evaluate the assertion.
 >>
 >> If there were only specific details that were relevant, only
 >> specific sections of the document would be cited. To evaluate the
 >> assertion read the cited documents. All of them. To understand
 >> this document, read and understand the normative references. All
 >> of them.
 >
 > Then that is what should be said in the specification.
 >
 > When citing references, it is typical (and I believe essential) to
 > have the citation text be specific about what it being used from the
 > cited document.
 >
 > If what is being used is 'everything', then that's what is said.
 > Explicitly. Language along the lines of "the current specification
 > requires completely familiarity with: cite1, cite2, ... is typical.

I've read a fair number of RFCs (though not all of them, by any means) 
but I do not recall ever having encountered that language.

>
 >
 >> My understanding, as expressed above, is that in order for a reader
 >> to expect to understand an RFC, they must first have read and
 >> understood the normative references. Is that incorrect?
 >
 > It depends upon what is being drawn from the references, and it is
 > the job of the referring text to explain its use of the reference.
 >
 > It also is not typical to require the reader to check the references
 > section to find out what is normative and what isn't. That's why we
 > have normative vocabulary.
 >
 > By requiring the reader to flip back and forth to the references
 > section, to find out what is normative and what isn't, the
 > specification is made markedly more cumbersome to use.

Having ascertained what the normative references are, having read and 
understood them, why would one need to flip back and forth?

>
 >
 >>> In general, this document's frequent use of "AVP" appears to be
 >>> an oddly syntactic focus. Typical specifications refer to
 >>> attributes, without such frequent, explicit reference to the form
 >>> of their having values. On its own, this point could seem like
 >>> quibbling, but it's part of the reaction I had when reading the
 >>> document, that is seemed less accessible than one would wish for
 >>> a new reader.
 >>
 >> This document is not intended to function as a primer on Diameter,
 >> AAA, or network access systems.
 >
 > Except that the document does provide /some/ primer material:
 >
 > The Diameter base protocol provides the following facilities: ... all
 > data delivered by the protocol is in the form of an AVP
 >
 >
 > Documents need to define their normative context. There is benefit
 > in making different documents minimize that context. Obviously
 > documents that extend a service must rely on the foundation of that
 > service, but the specification of an extension can be made more or
 > less complicated by the way divide-and-conquor writing is done.

Sorry, you've lost me.

>
 >
 >> The centrality of AVPs to the operation of the Diameter protocol
 >> might explain the focus of this draft upon them.
 >
 > My point was not that the use of attributes was unusual, but that the
 > terminologic focus on what is essentially a syntactic aspect of the
 > construct is unusual and, in my view, overly detailed. It's a bit
 > like spelling out 'period' at the end of every sentence in a
 > document. Distracting and unnecessary.
 >
 > And again, the term isn't defined in this document, in spite of there
 > being a Terminology section.

Again, the term is defined in RFC 3588.

>
 >
 >>>
 >>>
 >>>> by common usage. These are session identification,
 >>>> authentication, authorization, tunneling, and accounting. The
 >>>> authorization AVPs are further broken down by service type.
 >>>
 >>> This document appears to require deep knowledge of The Diameter
 >>> Base Protocol. In reality, I don't think this document can be
 >>> meaningfully read without that knowledge. So this document
 >>> should say that. (The language in the first paragraph of the
 >>> Intro doesn't actually say it.)
 >>
 >> One more time: read the normative references. Understand them.
 >> All of them.
 >
 > Once again: say that. have the text cite them as musts for
 > background and familiarity.

OK, let's try a thought experiment: suppose that I had included a 
(redundant) sentence like "The reader is assumed to have read and 
understood all of the documents listed as normative references." in the 
Abstract (& maybe, just to double-down on redundancy, in the 
Introduction, too).  Would you have actually done that as part of your 
review?

>
 >
 >>>> 1.6. Accounting Model
 >>>>
 >>>> It is RECOMMENDED that the coupled accounting model (Section
 >>>> 9.3 of [I-D.ietf-dime-rfc3588bis]) be used with this
 >>>> application; therefore, the value of the Acct-Application-Id
 >>>> AVP in the Accounting-Request (Section 3.10) and
 >>>> Accounting-Answer (Section 3.9) messages SHOULD be set to one
 >>>> (1).
 >>>
 >>> All of the values in those two sections (except the "271") are
 >>> symbolic. As such, setting a value to "1" has no obvious
 >>> meaning.
 >>>
 >>> I assume that the issue would be fixed by using symbolic values
 >>> here?
 >>
 >> Or by reading RFC 3588.
 >
 > Which part?

The part that is, in this case, specifically referenced: Section 9.3, 
reproduced below.

9.3.  Accounting Application Extension and Requirements

    Each Diameter application (e.g., NASREQ, Mobile IP) SHOULD define its
    service-specific AVPs that MUST be present in the Accounting-Request
    message in a section titled "Accounting AVPs".  The application MUST
    assume that the AVPs described in this document will be present in
    all Accounting messages, so only their respective service-specific
    AVPs need to be defined in that section.

    Applications have the option of using one or both of the following
    accounting application extension models:

    Split Accounting Service

       The accounting message will carry the Application Id of the
       Diameter base accounting application (see Section 2.4).
       Accounting messages may be routed to Diameter nodes other than the
       corresponding Diameter application.  These nodes might be
       centralized accounting servers that provide accounting service for
       multiple different Diameter applications.  These nodes MUST
       advertise the Diameter base accounting Application Id during
       capabilities exchange.


    Coupled Accounting Service

       The accounting message will carry the Application Id of the
       application that is using it.  The application itself will process
       the received accounting records or forward them to an accounting
       server.  There is no accounting application advertisement required
       during capabilities exchange, and the accounting messages will be
       routed the same way as any of the other application messages.

    In cases where an application does not define its own accounting
    service, it is preferred that the split accounting model be used.

Since the recommendation is to use the coupled accounting model, the 
value of the Acct-Application-Id AVP should be set to the NASREQ 
Application Id (1).

> in my review I tried to make  clear that I actually had
 > looked in RFC 3588 to resolve questions about the text I was
 > reviewing. Had that succeeded I would have made more specific
 > suggestions for resolving specific issues.
 >
 > So, for the current case, the word 'symbolic' appears in RFC 3588
 > exactly one time, for icmptypes types, very deep in Section 4.3. I
 > did not find it at all helpful for resolving the point I raised here
 > in the review.
 >
 >
 > Just to check, it appears the response to the review ended with this
 > item.

Yes, it was frankly very difficult to find anything that wasn't covered 
by either RTFRFC or a stylistic quibble.  I gave up.

>
 > d/