Re: [sipcore] location-conveyance-03 just submitted

"Thomson, Martin" <Martin.Thomson@andrew.com> Tue, 13 July 2010 05:53 UTC

Return-Path: <Martin.Thomson@andrew.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 2E4C03A6875 for <sipcore@core3.amsl.com>; Mon, 12 Jul 2010 22:53:49 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.719
X-Spam-Level:
X-Spam-Status: No, score=-2.719 tagged_above=-999 required=5 tests=[AWL=-0.120, BAYES_00=-2.599]
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 Wm7Fp7sRVh+I for <sipcore@core3.amsl.com>; Mon, 12 Jul 2010 22:53:45 -0700 (PDT)
Received: from csmailgw2.commscope.com (csmailgw2.commscope.com [198.135.207.242]) by core3.amsl.com (Postfix) with ESMTP id 263EC3A67C0 for <sipcore@ietf.org>; Mon, 12 Jul 2010 22:53:39 -0700 (PDT)
Received: from [10.86.20.103] ([10.86.20.103]:30134 "EHLO ACDCE7HC2.commscope.com") by csmailgw2.commscope.com with ESMTP id S337104Ab0GMFxr (ORCPT <rfc822; sipcore@ietf.org>); Tue, 13 Jul 2010 00:53:47 -0500
Received: from SISPE7HC1.commscope.com (10.97.4.12) by ACDCE7HC2.commscope.com (10.86.20.103) with Microsoft SMTP Server (TLS) id 8.1.436.0; Tue, 13 Jul 2010 00:53:47 -0500
Received: from SISPE7MB1.commscope.com ([fe80::9d82:a492:85e3:a293]) by SISPE7HC1.commscope.com ([fe80::8a9:4724:f6bb:3cdf%10]) with mapi; Tue, 13 Jul 2010 13:53:45 +0800
From: "Thomson, Martin" <Martin.Thomson@andrew.com>
To: "James M. Polk" <jmpolk@cisco.com>, "sipcore@ietf.org" <sipcore@ietf.org>
Date: Tue, 13 Jul 2010 13:55:51 +0800
Thread-Topic: [sipcore] location-conveyance-03 just submitted
Thread-Index: AcsiHaubmUNEswfFSNG6YyIwHpshjQABipyw
Message-ID: <8B0A9FCBB9832F43971E38010638454F03E9DCCFFC@SISPE7MB1.commscope.com>
References: <201007122355.o6CNt6us024310@sj-core-3.cisco.com>
In-Reply-To: <201007122355.o6CNt6us024310@sj-core-3.cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-BCN: Meridius 1000 Version 3.4 on csmailgw2.commscope.com
X-BCN-Sender: Martin.Thomson@andrew.com
Subject: Re: [sipcore] location-conveyance-03 just submitted
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: Tue, 13 Jul 2010 05:53:52 -0000

Firstly, the good:

The simplifications are a great improvement.  The document is more accessible and readable as a result.  The mechanism is fundamentally simpler.

Content indirection has some interesting benefits.  It's a good idea.

I have some comments that includes one major issue.

The major issue:

The limitation to _one_ location value has undeniable benefits.  It also prevents the conveyance of location value and reference together.

draft-ietf-ecrit-rough-loc describes one use case that this affects.  Brian (your other co-author) should be able to apprise you of the consequences this has to the NENA use cases.

Jon's original response to this was to push the composition problem to the PIDF-LO.  That could be done, but it needs more specification.

Reaching some sort of consensus on the solution to this problem before this work is finished is important.

Minor issues:

The structure of the Geolocation-Error header description is a little odd.  It seems that you are trying to describe *classes* of error and specific errors at the same time.  Separating these concerns might make this easier to explain and understand.

The draft doesn’t really justify the existence of the "geolocation" option tag.

Section 4.6 should reference draft-ietf-geopriv-deref-protocol.

Section 5 should show an example use of content indirection for a location URI.

Just a few nits:

Overall, the structure is greatly improved, but when you see references in the form "in section 3 (below Figure 4)", you have a need for more sub-sections.

The examples in Section 5 are not schema-valid - c.f. RFC 4119 errata #1771
  <http://www.rfc-editor.org/errata_search.php?rfc=4119>

This, in the introduction, doesn't read well:

  a non-IP device (a person or a black phone)

"black phone" is a little jargon-y.  More amusing was the idea that your characterisation of a person was as a "non-IP device".

--Martin

> -----Original Message-----
> From: sipcore-bounces@ietf.org [mailto:sipcore-bounces@ietf.org] On
> Behalf Of James M. Polk
> Sent: Tuesday, 13 July 2010 9:55 AM
> To: sipcore@ietf.org
> Subject: [sipcore] location-conveyance-03 just submitted
> 
> SIPCORE
> 
> http://www.ietf.org/internet-drafts/draft-ietf-sipcore-location-
> conveyance-03.txt
> 
> This version is a fairly radical departure from previous versions of
> the (long standing) effort.  This time the doc didn't grow...
> 
> We added Jon Peterson as a co-author, and he helped in Anaheim
> propose a significantly less complex way of specifying location
> conveyance for SIP. More than 95% of the doc has been rewritten by
> Jon and I, with a net result of 27 less pages of text (52 to 25 now).
> 
> We took out the terms and concepts of following parameters:
> 
> - inserter
> - inserted-by
> - used-for-routing
> - host-id
> - node-id
> 
> as well as limiting the number of locationValues to *ONE*, in a SIP
> request.
> 
> We removed the ability for SIP intermediaries to add location along
> the way unbeknownst to the UAC (with one infrequently used
> exception). In this way, location errors are only end-to-end, so
> there is no need to identify which entity added location to make sure
> the right entity reacted the right way when an error was caused by
> the location they inserted, and not that of another entity's inserted
> location (which is all very confusing).
> 
> We added the ability for a SIP intermediary to augment a location
> with what the UAC should send in a subsequent request, and the
> ability to verify this augmentation is present before successfully
> processing that request. This is in line with both RFCs 4479 and 5491.
> 
> Comments are appreciated
> 
> James
> 
> _______________________________________________
> sipcore mailing list
> sipcore@ietf.org
> https://www.ietf.org/mailman/listinfo/sipcore