Re: [martini] Minor comments on temp-gruus in gin-07

Adam Roach <adam@nostrum.com> Mon, 27 September 2010 16:52 UTC

Return-Path: <adam@nostrum.com>
X-Original-To: martini@core3.amsl.com
Delivered-To: martini@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 5B91D3A6D98 for <martini@core3.amsl.com>; Mon, 27 Sep 2010 09:52:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.678
X-Spam-Level:
X-Spam-Status: No, score=-102.678 tagged_above=-999 required=5 tests=[AWL=-0.079, BAYES_00=-2.599, 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 0SO0cTlHuPP1 for <martini@core3.amsl.com>; Mon, 27 Sep 2010 09:51:56 -0700 (PDT)
Received: from nostrum.com (nostrum-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:267::2]) by core3.amsl.com (Postfix) with ESMTP id A20633A6D74 for <martini@ietf.org>; Mon, 27 Sep 2010 09:51:55 -0700 (PDT)
Received: from dn3-228.estacado.net (vicuna-alt.estacado.net [75.53.54.121]) (authenticated bits=0) by nostrum.com (8.14.3/8.14.3) with ESMTP id o8RGqWo9061500 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO); Mon, 27 Sep 2010 11:52:33 -0500 (CDT) (envelope-from adam@nostrum.com)
Message-ID: <4CA0CBD1.3010905@nostrum.com>
Date: Mon, 27 Sep 2010 11:52:33 -0500
From: Adam Roach <adam@nostrum.com>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.9) Gecko/20100915 Thunderbird/3.1.4
MIME-Version: 1.0
To: "Elwell, John" <john.elwell@siemens-enterprise.com>
References: <A444A0F8084434499206E78C106220CA01C81C2A34@MCHP058A.global-ad.net>
In-Reply-To: <A444A0F8084434499206E78C106220CA01C81C2A34@MCHP058A.global-ad.net>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: "martini@ietf.org" <martini@ietf.org>
Subject: Re: [martini] Minor comments on temp-gruus in gin-07
X-BeenThere: martini@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: Discussion of en-mass SIP PBX registration mechanisms <martini.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/martini>, <mailto:martini-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/martini>
List-Post: <mailto:martini@ietf.org>
List-Help: <mailto:martini-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/martini>, <mailto:martini-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Sep 2010 16:52:02 -0000

  Thanks, John. I agree with both of these comments (the change from 
"must" to "MUST" was an error on my part when I went through and tried 
to fix the normative language elsewhere). I need to fix a typo or two in 
-07 anyway, so I'll roll these changes into the next edit.

/a

On 9/27/10 11:07 AM, Elwell, John wrote:
> 1. In 7.1.2:
> "The SIP-PBX MUST be able to construct a temp-gruu in a way that the
>     SSP can decode."
> In 05, it used to be "must". I think this is only intended to be an informative statement, as part of the justification why we need to standardize a mechanism. I don't think this section is placing requirements on the SIP-PBX - that should be the job of 7.1.2.2. I think "must" (or "needs to") is correct.
>
> 2. In 7.1.2.2:
> "SIP-
>         PBXes that implement temporary GRUUs MUST generate a new temp-gruu
>         according to the procedures in this section for every registration
>         refresh. "
> It is unclear, but I think this is intended to apply to temp gruus that the SIP-PBX provides to its registering UAs, and applies to initial registrations and registration refreshes by to those UAs (as opposed to registration refreshes with the SP). Just to make it more explicit, how about:
> "SIP-
>         PBXes that implement temporary GRUUs MUST generate a new temp-gruu
>         according to the procedures in this section for every registration
>         or registration refresh from GRUU-supporting UAs attached to the SIP-PBX. "
>
> John
> _______________________________________________
> martini mailing list
> martini@ietf.org
> https://www.ietf.org/mailman/listinfo/martini