[apps-discuss] Apps-team review of draft-ietf-sipcore-reinvite-07

Enrico Marocco <enrico.marocco@telecomitalia.it> Mon, 06 December 2010 14:35 UTC

Return-Path: <enrico.marocco@telecomitalia.it>
X-Original-To: apps-discuss@core3.amsl.com
Delivered-To: apps-discuss@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 722FD3A67F9 for <apps-discuss@core3.amsl.com>; Mon, 6 Dec 2010 06:35:42 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -100.223
X-Spam-Level:
X-Spam-Status: No, score=-100.223 tagged_above=-999 required=5 tests=[AWL=0.496, BAYES_00=-2.599, HELO_EQ_IT=0.635, HOST_EQ_IT=1.245, USER_IN_WHITELIST=-100]
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 GT9FuN6fdQgq for <apps-discuss@core3.amsl.com>; Mon, 6 Dec 2010 06:35:41 -0800 (PST)
Received: from GRFEDG702BA020.telecomitalia.it (grfedg702ba020.telecomitalia.it [156.54.233.201]) by core3.amsl.com (Postfix) with ESMTP id 26AC63A67B4 for <apps-discuss@ietf.org>; Mon, 6 Dec 2010 06:35:41 -0800 (PST)
Received: from GRFHUB701BA020.griffon.local (10.188.101.111) by GRFEDG702BA020.telecomitalia.it (10.188.45.101) with Microsoft SMTP Server (TLS) id 8.2.254.0; Mon, 6 Dec 2010 15:37:01 +0100
Received: from pc-moris.cselt.it (163.162.173.6) by smtp.telecomitalia.it (10.188.101.114) with Microsoft SMTP Server (TLS) id 8.2.254.0; Mon, 6 Dec 2010 15:36:58 +0100
Message-ID: <4CFCF509.4000607@telecomitalia.it>
Date: Mon, 06 Dec 2010 15:36:57 +0100
From: Enrico Marocco <enrico.marocco@telecomitalia.it>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.12) Gecko/20101027 Thunderbird/3.1.6
MIME-Version: 1.0
To: draft-ietf-sipcore-reinvite.all@tools.ietf.org
Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg="sha1"; boundary="------------ms080605030105000708070809"
Cc: "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Subject: [apps-discuss] Apps-team review of draft-ietf-sipcore-reinvite-07
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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: Mon, 06 Dec 2010 14:35:42 -0000

I have been selected as the Applications Area Review Team reviewer for
this draft (for background on apps-review, please see
http://www.apps.ietf.org/content/applications-area-review-team).
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-sipcore-reinvite-07
Title: Re-INVITE and Target-refresh Request Handling in the Session
Initiation Protocol (SIP)
Reviewer: Enrico Marocco
Review Date: Dec 6, 2010

Summary: This draft is basically ready for publication as an
Informational RFC. A few clarifications suggested as minor issues.

Major Issues: none.

Minor Issues:

In S. 4.1., the definition of local target is formally correct, but
quite hard to parse. A simpler wording or maybe an example could save
several brain cycles.

In S. 4.4., last paragraph mixes normative and non-normative terminology
and does not provide enough information for determining when mixing
target refresh and media session changes may be considered acceptable.

Nits: no typos found.