Re: [sipcore] location conveyance looks good

"James M. Polk" <jmpolk@cisco.com> Wed, 27 October 2010 04:28 UTC

Return-Path: <jmpolk@cisco.com>
X-Original-To: sipcore@core3.amsl.com
Delivered-To: sipcore@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 3F4793A67D3 for <sipcore@core3.amsl.com>; Tue, 26 Oct 2010 21:28:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -110.542
X-Spam-Level:
X-Spam-Status: No, score=-110.542 tagged_above=-999 required=5 tests=[AWL=0.057, BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, 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 wSqFW3oH9F-t for <sipcore@core3.amsl.com>; Tue, 26 Oct 2010 21:28:10 -0700 (PDT)
Received: from sj-iport-6.cisco.com (sj-iport-6.cisco.com [171.71.176.117]) by core3.amsl.com (Postfix) with ESMTP id D667B3A6992 for <sipcore@ietf.org>; Tue, 26 Oct 2010 21:28:08 -0700 (PDT)
Authentication-Results: sj-iport-6.cisco.com; dkim=neutral (message not signed) header.i=none
X-IronPort-AV: E=Sophos;i="4.58,244,1286150400"; d="scan'208";a="610238443"
Received: from sj-core-2.cisco.com ([171.71.177.254]) by sj-iport-6.cisco.com with ESMTP; 27 Oct 2010 04:29:57 +0000
Received: from jmpolk-wxp01.cisco.com (rcdn-jmpolk-8715.cisco.com [10.99.80.22]) by sj-core-2.cisco.com (8.13.8/8.14.3) with ESMTP id o9R4TuCM027015; Wed, 27 Oct 2010 04:29:56 GMT
Message-Id: <201010270429.o9R4TuCM027015@sj-core-2.cisco.com>
X-Mailer: QUALCOMM Windows Eudora Version 7.1.0.9
Date: Tue, 26 Oct 2010 23:29:55 -0500
To: "Thomson, Martin" <Martin.Thomson@andrew.com>, "James M. Polk" <jmpolk@cisco.com>, "Peterson, Jon" <jon.peterson@neustar.biz>
From: "James M. Polk" <jmpolk@cisco.com>
In-Reply-To: <8B0A9FCBB9832F43971E38010638454F03F31EAF78@SISPE7MB1.comms cope.com>
References: <8B0A9FCBB9832F43971E38010638454F03F31EAF78@SISPE7MB1.commscope.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; format=flowed
Cc: "sipcore@ietf.org" <sipcore@ietf.org>
Subject: Re: [sipcore] location conveyance looks good
X-BeenThere: sipcore@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: SIP Core Working Group <sipcore.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/sipcore>, <mailto:sipcore-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sipcore>
List-Post: <mailto:sipcore@ietf.org>
List-Help: <mailto:sipcore-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sipcore>, <mailto:sipcore-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 27 Oct 2010 04:28:12 -0000

At 12:50 AM 10/26/2010, Thomson, Martin wrote:
>Reviewing the changes in 
>draft-ietf-sipcore-location-conveyance-04.  Nothing much to add except nits.
>
>I could quibble.  I still don't like or want the entirety of Section 4.3

that would be quibbling, and we don't want that, now do we?

>and the serial nature Figure 2 is a little mysterious to me

if I send you a SIP request with a location URI, you need to be able 
to dereference it (or at least try) before you accept the SIP 
request. I know it'll add a bit of time to the 200 OK, but that will 
allow confirmation of location delievery - which isn't the location 
URI, but the location value itself (i.e., the PIDF-LO). If the 
dereference fails after you've sent me a 200 OK, how can you possibly 
tell me that there was something wrong with my delivery of location to you?

>, but it's down to subjective stuff like that, the effort just 
>doesn't pay off.
>
>So, ship it.
>
>--Martin
>
>
>Nits:
>
>This seems odd, especially since the parameter is MUST-strength mandatory:
>
>    ...  If no routing-allowed parameter
>    is present in a SIP request, a SIP intermediary MAY insert this
>    value with a RECOMMENDED value of "no" by default.
>
>Cross references should capitalize the 's' in Section 3.4.
>
>The MIME multipart headers are still in the wrong place (see Section 
>5 of RFC 2387 to see how this is done):
>
>OLD:
>--boundary
>
>Content-Type: application/pidf+xml
>Content-ID: <blahblah@example.com>
><?xml version="1.0" standalone="true"?>
><presence ...
>
>NEW:
>--boundary
>Content-Type: application/pidf+xml
>Content-ID: <blahblah@example.com>
>
><?xml version="1.0" standalone="true"?>
><presence ...