[apps-discuss] AppsDir review of draft-ietf-sipclf-format-05

Claudio Allocchio <Claudio.Allocchio@garr.it> Thu, 19 January 2012 10:21 UTC

Return-Path: <Claudio.Allocchio@garr.it>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9972421F846A for <apps-discuss@ietfa.amsl.com>; Thu, 19 Jan 2012 02:21:41 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.449
X-Spam-Level:
X-Spam-Status: No, score=-1.449 tagged_above=-999 required=5 tests=[AWL=-1.150, BAYES_00=-2.599, MANGLED_DEALS=2.3]
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 cWBO7v1d+fwH for <apps-discuss@ietfa.amsl.com>; Thu, 19 Jan 2012 02:21:41 -0800 (PST)
Received: from cyrus.dir.garr.it (cyrus.dir.garr.it [IPv6:2001:760:0:158::29]) by ietfa.amsl.com (Postfix) with ESMTP id 514C021F8469 for <apps-discuss@ietf.org>; Thu, 19 Jan 2012 02:21:35 -0800 (PST)
Received: from mac-allocchio3.elettra.trieste.it (mac-allocchio3.elettra.trieste.it [140.105.2.18]) (authenticated bits=0) by cyrus.dir.garr.it (8.14.5/8.14.5) with ESMTP id q0JALVN0014638 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 19 Jan 2012 11:21:32 +0100 (CET)
X-DomainKeys: Sendmail DomainKeys Filter v1.0.2 cyrus.dir.garr.it q0JALVN0014638
DomainKey-Signature: a=rsa-sha1; s=mail; d=garr.it; c=simple; q=dns; b=F0KwaOzwcUMxTMMb5s2AIu9T0c3SojcBDMVq5WZOPdJcmhWCXRH8QQLkekL9HeisA GOT5ue0Ks0OnElyUSAECTcjE1EMmVH4Ie2KNE5xywpnYx+GI6CIxPsnex2nfw/d55Tm xvnKIvLtdTpMLLRf9Sm+HNWrvly2hnqbAlJ9bLI=
Date: Thu, 19 Jan 2012 11:21:29 +0100 (CET)
From: Claudio Allocchio <Claudio.Allocchio@garr.it>
X-X-Sender: claudio@mac-allocchio3.elettra.trieste.it
To: apps-discuss@ietf.org, draft-ietf-sipclf-format.all@tools.ietf.org
Message-ID: <alpine.OSX.2.02.1201161423200.36734@mac-allocchio3.garrtest.units.it>
User-Agent: Alpine 2.02 (OSX 1266 2009-07-14)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; format=flowed; charset=US-ASCII
Subject: [apps-discuss] AppsDir review of draft-ietf-sipclf-format-05
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 19 Jan 2012 10:21:41 -0000

I have been selected as the Applications Area Directorate reviewer for 
this draft (for background on appsdir, please see 
http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate).

Please resolve these comments along with any other Last Call comments you 
may receive. Please wait for direction from your document shepherd or AD 
before posting a new version of the draft.

Document: draft-ietf-sipclf-format-05
Title: Format for the Session Initiation Protocol (SIP) Common Log Format
        (CLF)
Reviewer: Claudio Allocchio
Review Date: 2012-01-18
IETF Last Call Date: unknown
IESG Telechat Date: unknown

NOTE: is there is an IESG telechat which I'm missing, just copy this 
messae to iesg ML!

Summary:

This draft is ready for publication as a Proposed Standard RFC; it just 
needs one consideration about a minor issue about an external reference, 
and fixing some Nits.

Major Issues:

NONE

Minor Issues:

just one !

* Section "3. Document Conventions"

The authors have decided "For the sake of clarity and completeness" to 
quote a full section of RFC4475 into this document. However there were 
quite long discussions in the Apps Dir in other reviews if this quoting 
practice is appropriate or not, because it can lead to discrepacies, when 
the quoted document is updated or obsoleted, resuing in confusion for the 
readers who just trust the quoted text without checking the state of the 
referenced document. I would suggest, as in the other cases, NOT to quote 
the external text, but to use an explicit external reference only, urging 
the reader to check that "for the sake of clarity".

Nits:

* Section "Copyright Notice"

change the year to 2012.

* Section "3. Document Conventions"

remember to change "formatting rules for Internet-Drafts" into "formatting 
rules for RFCs" when published.

* Section "4. Format"

I do understand that it probably does not fit into the I-D formatting line 
lenghts, but... maybe an attempt to add aside the SIP CLF record depicted 
in Figure 1 the grouping described as

                                  <IndexPointers>
                                  <MandatoryFields>
                                  <OptionalFields>

in the figure 1 itself (on the left?) may help in reading the distinction, 
and maybe may even not require to repeat portions of the figure itself 
later in the I-D, like in figure 3, figure 4 and figure 5. It is really a 
nit and just a suggestion. Not really important.

What I am suggesting:

        +----------------------------------+
       /|                                  |
      / |                                  |
    P   |                                  |
    o   |                                  |
  I i   |                                  |
  n n   |                                  |
  d t   |                                  |
  e e   |                                  |
  x r   |                                  |
    s   |                                  |
     \  |                                  |
       \|                                  |
        +----------------------------------+
       /|                                  |
      / |                                  |
  M     |                                  |
  a F   |                                  |
  n i   |                                  |
  d e   |                                  |
  a l   |                                  |
  t d   |                                  |
  o s   |                                  |
  r     |                                  |
  y  \  |                                  |
       \|                                  |
        +----------------------------------+
       /|                                  |
      / |                                  |
  O     |                                  |
  p F   |                                  |
  t i   |                                  |
  i e   |                                  |
  o l   |                                  |
  n d   |                                  |
  a s   |                                  |
  l     |                                  |
     \  |                                  |
       \|                                  |
        +----------------------------------+

* Section "5. Example SIP CLF Record"

Remember to change "Due to internet-draft conventions" into "Due to RFC 
conventions" when published (and also "to format it for an RFC" later 
on...).

* From the DataTracker ID Nits check tool (References Nits):

   == Unused Reference: 'RFC5612' is defined on line 984, but no explicit
      reference was found in the text

   ** Downref: Normative reference to an Informational draft:
      draft-ietf-sipclf-problem-statement (ref.
      'I-D.ietf-sipclf-problem-statement')

Comment: the document shepard is aware and has a solution for these nits. 
Just apply it.

------------------------------------------------------------------------------
Claudio Allocchio             G   A   R   R          Claudio.Allocchio@garr.it
                         Senior Technical Officer
tel: +39 040 3758523      Italian Academic and       G=Claudio; S=Allocchio;
fax: +39 040 3758565        Research Network         P=garr; A=garr; C=it;

            PGP Key: http://www.cert.garr.it/PGP/keys.php3#ca