Gen-ART review of draft-ietf-sipping-sbc-funcs-05.txt

Black_David@emc.com Fri, 04 April 2008 14:33 UTC

Return-Path: <ietf-bounces@ietf.org>
X-Original-To: ietf-archive@megatron.ietf.org
Delivered-To: ietfarch-ietf-archive@core3.amsl.com
Received: from core3.amsl.com (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id AAED43A6E72; Fri, 4 Apr 2008 07:33:06 -0700 (PDT)
X-Original-To: ietf@core3.amsl.com
Delivered-To: ietf@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id CF1093A6C4C; Fri, 4 Apr 2008 07:02:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.288
X-Spam-Level:
X-Spam-Status: No, score=-4.288 tagged_above=-999 required=5 tests=[AWL=2.311, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id SXc-K8P3iWMt; Fri, 4 Apr 2008 07:02:28 -0700 (PDT)
Received: from mexforward.lss.emc.com (mexforward.lss.emc.com [128.222.32.20]) by core3.amsl.com (Postfix) with ESMTP id 6DFB028C40C; Fri, 4 Apr 2008 07:02:21 -0700 (PDT)
Received: from hop04-l1d11-si03.isus.emc.com (HOP04-L1D11-SI03.isus.emc.com [10.254.111.23]) by mexforward.lss.emc.com (Switch-3.2.5/Switch-3.1.7) with ESMTP id m34E2L7m011804; Fri, 4 Apr 2008 10:02:21 -0400 (EDT)
Received: from mailhub.lss.emc.com (uraeus.lss.emc.com [10.254.144.14]) by hop04-l1d11-si03.isus.emc.com (Tablus Interceptor); Fri, 4 Apr 2008 11:57:47 -0400
Received: from corpussmtp4.corp.emc.com (corpussmtp4.corp.emc.com [10.254.64.54]) by mailhub.lss.emc.com (Switch-3.2.5/Switch-3.1.7) with ESMTP id m34E27Fh002163; Fri, 4 Apr 2008 10:02:13 -0400 (EDT)
From: Black_David@emc.com
Received: from CORPUSMX20A.corp.emc.com ([128.221.62.11]) by corpussmtp4.corp.emc.com with Microsoft SMTPSVC(6.0.3790.1830); Fri, 4 Apr 2008 10:02:06 -0400
X-MimeOLE: Produced By Microsoft Exchange V6.5
Content-class: urn:content-classes:message
MIME-Version: 1.0
Subject: Gen-ART review of draft-ietf-sipping-sbc-funcs-05.txt
Date: Fri, 04 Apr 2008 10:02:02 -0400
Message-ID: <8CC6CEAB44F131478D3A7B429ECACD91016F5F02@CORPUSMX20A.corp.emc.com>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: Gen-ART review of draft-ietf-sipping-sbc-funcs-05.txt
Thread-Index: AciWXHUGeMgU94roTG6JPzmBP74law==
To: Jani.Hautakorpi@ericsson.com, Gonzalo.Camarillo@ericsson.com, bpenfield@acmepacket.com, alan.ietf@polyphase.ca, mbhatia@3clogic.com, gen-art@ietf.org, sipping@ietf.org, ietf@ietf.org
X-OriginalArrivalTime: 04 Apr 2008 14:02:06.0372 (UTC) FILETIME=[775BC640:01C8965C]
X-PMX-Version: 4.7.1.128075, Antispam-Engine: 2.5.1.298604, Antispam-Data: 2007.8.30.51425
X-PerlMx-Spam: Gauge=, SPAM=0%, Reason='EMC_BODY_1+ -3, EMC_FROM_0+ -3, NO_REAL_NAME 0, __C230066_P5 0, __CP_NOT_1 0, __CP_URI_IN_BODY 0, __CT 0, __CTE 0, __CT_TEXT_PLAIN 0, __HAS_MSGID 0, __IMS_MSGID 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __SANE_MSGID 0'
X-Tablus-Inspected: yes
X-Tablus-Classifications: public
X-Tablus-Action: allow
X-Mailman-Approved-At: Fri, 04 Apr 2008 07:33:05 -0700
Cc: Black_David@emc.com, jon.peterson@neustar.biz, mary.barnes@nortel.com
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: ietf-bounces@ietf.org
Errors-To: ietf-bounces@ietf.org

I have been selected as the General Area Review Team (Gen-ART) 
reviewer for this draft (for background on Gen-ART, please see 
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). 

Please wait for direction from your document shepherd 
or AD before posting a new version of the draft. 

Document: draft-ietf-sipping-sbc-funcs-05.txt
Reviewer: David L. Black
Review Date: 4 April 2008
IESG Telechat date: 10 April 2008 

Summary:
This draft is basically ready for publication, but has nits that
should be fixed before publication.

Comments:
The authors have addressed all of the comments from the Gen-ART
review of the -04 version of this draft - that was nicely done,
and many thanks to the authors.

The draft could be published as an RFC in its current form, but
there's one nit that really should be dealt with:

One of the comments was addressed by adding the last sentence
to this paragraph in Section 3.4.2:

   There is also a problem related to the method how SBCs choose the
   value for the validity of a registration period.  This value should
   be as high as possible, but it still needs to be low enough to
   maintain the NAT binding.  Typically SBCs do not have any
   deterministic method for choosing a suitable value.  However, SBCs
   can just use a sub-optimal, relatively small value which usually
   works.

Please provide an example of "a sub-optimal, relatively small value
which usually works".  This merits careful consideration, as
implementers may well use the value provided as an example.  One
possibility is to use 15 seconds as the example and cite Section 3.5
of draft-ietf-tsvwg-udp-guidelines-06.txt (as an informative
reference) for the source of this value (and a useful place to
look for further discussion of this topic).

Minor nit: Section 3.1.3:

   (i.e., information related to network elements is beeing hidden),
Extra "e" here ------------------------------------> ^^^^^^

idnits 2.08.05 ran clean.

Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953             FAX: +1 (508) 293-7786
black_david@emc.com        Mobile: +1 (978) 394-7754
----------------------------------------------------

_______________________________________________
IETF mailing list
IETF@ietf.org
https://www.ietf.org/mailman/listinfo/ietf