[apps-discuss] APPSDIR review of draft-ietf-straw-sip-traceroute-02
"Vijay K. Gurbani" <vkg@bell-labs.com> Tue, 24 June 2014 14:47 UTC
Return-Path: <vkg@bell-labs.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BB7631B2AAE; Tue, 24 Jun 2014 07:47:06 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_50=0.8, RCVD_IN_DNSWL_HI=-5] autolearn=ham
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 bcxug9kdo6d2; Tue, 24 Jun 2014 07:47:02 -0700 (PDT)
Received: from ihemail1.lucent.com (ihemail1.lucent.com [135.245.0.33]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 363291B2A4E; Tue, 24 Jun 2014 07:46:58 -0700 (PDT)
Received: from usnavsmail1.ndc.alcatel-lucent.com (usnavsmail1.ndc.alcatel-lucent.com [135.3.39.9]) by ihemail1.lucent.com (8.13.8/IER-o) with ESMTP id s5OEkuDG029259 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Tue, 24 Jun 2014 09:46:56 -0500 (CDT)
Received: from umail.lucent.com (umail.ndc.lucent.com [135.3.40.61]) by usnavsmail1.ndc.alcatel-lucent.com (8.14.3/8.14.3/GMO) with ESMTP id s5OEktwY007522 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Tue, 24 Jun 2014 09:46:56 -0500
Received: from shoonya.ih.lucent.com (shoonya.ih.lucent.com [135.185.238.169]) by umail.lucent.com (8.13.8/TPES) with ESMTP id s5OEkt3f012082; Tue, 24 Jun 2014 09:46:55 -0500 (CDT)
Message-ID: <53A98FF9.1050802@bell-labs.com>
Date: Tue, 24 Jun 2014 09:49:29 -0500
From: "Vijay K. Gurbani" <vkg@bell-labs.com>
Organization: Bell Laboratories, Alcatel-Lucent
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:24.0) Gecko/20100101 Thunderbird/24.5.0
MIME-Version: 1.0
To: "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Scanned-By: MIMEDefang 2.64 on 135.3.39.9
Archived-At: http://mailarchive.ietf.org/arch/msg/apps-discuss/X4wDihyq_mX41LrWfC_V3reR5t8
Cc: draft-ietf-straw-sip-traceroute.all@tools.ietf.org, IESG IESG <iesg@ietf.org>
Subject: [apps-discuss] APPSDIR review of draft-ietf-straw-sip-traceroute-02
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.15
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: Tue, 24 Jun 2014 14:47:06 -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-straw-sip-traceroute-02 Title: A Media-based Traceroute Function for the Session Initiation Protocol (SIP) Reviewer: Vijay K. Gurbani Review Date: Jun 24-2014 IETF Last Call Date: Jul-04-2014 IESG Telechat Date: Jul-10-2014 Summary: This draft is ready for publication as a Proposed Standard. Some nits and a confirmation (1 minor issue) below will require some attention. Major: 0 Minor: 1 Nits: Various Minor: - Just a confirmation: we are not defining a Required or Supported token for the traceroute mechanism described in the draft? S3, second paragraph states that, "or if the next-hop is a B2BUA that supports this mechanism, and if the B2BUA allows such testing from the requesting UAC," Here, our implicit assumption is that a B2BUA functionality will be upgraded automatically such that if it sees a M-F of 0 it create a media loopback session. If the B2BUA does not support creation of media loopback sessions, it simply sends a 483. In addition, B2BUAs that support creation of media streams MUST insert a SIP Reason header field cause value of 483. It seems that the support of the traceroute mechanisms requires some non-trivial steps by the B2BUA. But we are doing this without requiring any tokens in Required or Supported header and depending on configured policies (c.f., S3.2). Yes? - S3, third paragraph: Before creating another INV with a M-F of 1, the UAC should tear down the previous session. This is implicit behaviour of any sane implementation course, but should be stated nonetheless. Nits: - Abstract: s/field, in order to/field to/ s/generated which result/generated that result/ s/mechanism, in order to test/mechanism to/ - S2: s/rich-ringtones/rich ringtones/ s/As more and more SIP domains get deployed and interconnect/As increasingly more SIP domains get deployed and interconnected/ s/If failures or degradation occur in the media plane, it is difficult to determine where in the media path they occur./When a failure or degradation occurs in the media plane, it is difficult to determine where in the media path it occurred./ s/and media crossing/and media traversing/ s/to be able to progressively test/to progressively test/ s/will be crossed/will be traversed/ s/what URIs to/which URIs to/ -S3: s/originating UAC generates/originating UAC (Alice) generates/ s/a target AoR/a target AoR (Bob)/ Thanks, - vijay -- Vijay K. Gurbani, Bell Laboratories, Alcatel-Lucent 1960 Lucent Lane, Rm. 9C-533, Naperville, Illinois 60563 (USA) Email: vkg@{bell-labs.com,acm.org} / vijay.gurbani@alcatel-lucent.com Web: http://ect.bell-labs.com/who/vkg/ | Calendar: http://goo.gl/x3Ogq
- [apps-discuss] APPSDIR review of draft-ietf-straw… Vijay K. Gurbani