Re: Gen-ART Last Call review of draft-ietf-cuss-sip-uui-reqs-06

Alan Johnston <alan.b.johnston@gmail.com> Thu, 27 October 2011 16:35 UTC

Return-Path: <alan.b.johnston@gmail.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 00D3521F8B55 for <ietf@ietfa.amsl.com>; Thu, 27 Oct 2011 09:35:22 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.371
X-Spam-Level:
X-Spam-Status: No, score=-103.371 tagged_above=-999 required=5 tests=[AWL=-0.000, BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-1, SARE_SUB_OBFU_Q1=0.227, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id COsx-CKCR+Vd for <ietf@ietfa.amsl.com>; Thu, 27 Oct 2011 09:35:21 -0700 (PDT)
Received: from mail-qw0-f44.google.com (mail-qw0-f44.google.com [209.85.216.44]) by ietfa.amsl.com (Postfix) with ESMTP id D92DA21F8B06 for <ietf@ietf.org>; Thu, 27 Oct 2011 09:35:20 -0700 (PDT)
Received: by qadc10 with SMTP id c10so3494683qad.31 for <ietf@ietf.org>; Thu, 27 Oct 2011 09:35:20 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=from:content-type:subject:date:references:to:message-id :mime-version:x-mailer; bh=Y3tH9F6WT/r3/hglcyKOwS2N4i1l4fw/euJX9McMUa0=; b=I2BQypNxXVtN6J++OM/AywpW6aUsxepxz5RzwyJWtGiSwLajG5B3g8zj+NLkgx+PMN LVpDKO/UCgTYr8o+5WbFYksqzg8t+ftdwmtwkv1bzZ0TLcPRwxrN7JNlI4l7r2VHnwrV 02+RTSesh66pMq2Z+GlG9l9C1nXaXkGDPDvNo=
Received: by 10.224.76.209 with SMTP id d17mr30343875qak.87.1319733320385; Thu, 27 Oct 2011 09:35:20 -0700 (PDT)
Received: from [172.17.125.2] ([12.238.236.2]) by mx.google.com with ESMTPS id z17sm8939421qap.19.2011.10.27.09.35.18 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 27 Oct 2011 09:35:19 -0700 (PDT)
From: Alan Johnston <alan.b.johnston@gmail.com>
Content-Type: multipart/alternative; boundary="Apple-Mail-8--23778515"
Subject: Re: Gen-ART Last Call review of draft-ietf-cuss-sip-uui-reqs-06
Date: Thu, 27 Oct 2011 11:35:21 -0500
References: <CAEDAC25-7873-4DC7-9B93-C1EB1951FF9E@gmail.com>
To: ietf@ietf.org
Message-Id: <C33EEA2C-E88C-4206-81D7-9F3ECD463719@gmail.com>
Mime-Version: 1.0 (Apple Message framework v1084)
X-Mailer: Apple Mail (2.1084)
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 27 Oct 2011 16:35:22 -0000

Begin forwarded message:

> From: Alan Johnston <alan.b.johnston@gmail.com>
> Date: October 27, 2011 10:53:27 AM CDT
> To: Ben Campbell <ben@nostrum.com>
> Cc: draft-ietf-cuss-sip-uui-reqs.all@tools.ietf.org, "gen-art@ietf.org Review Team" <gen-art@ietf.org>, "Cuss@ietf.org" <cuss@ietf.org>
> Subject: Re: Gen-ART Last Call review of draft-ietf-cuss-sip-uui-reqs-06
> 
> Ben,
> 
> Thanks for your review of the draft.  See my comments below.  I have also revised the draft as draft-ietf-cuss-sip-uui-reqs-07 which I  believe addresses all the issues.
> 
> - Alan -
> 
> On Oct 12, 2011, at 4:27 PM, Ben Campbell wrote:
> 
>> I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>> 
>> Please resolve these comments along with any other Last Call comments you may receive.
>> 
>> Document: draft-ietf-cuss-sip-uui-reqs-06
>> Reviewer: Ben Campbell
>> Review Date: 2011-10-12
>> IETF LC End Date: 2011-10-13
>> 
>> Summary: This draft is almost ready for publication as an informational RFC. I have a few minor questions and comments that may be worth addressing first.
>> 
>> Major issues:
>> 
>> None
>> 
>> Minor issues:
>> 
>> -- section 1, 2nd paragraph, last sentence: "In particular, this mechanism creates no requirements on intermediaries such as proxies."
>> 
>> What about SBCs, B2BUAs, etc?
> 
> There are no requirements on them either - we'll add text saying that.
> 
>> 
>> -- REQ-4: "… any other form of redirection of the request."
>> 
>> "Any other form" seems a pretty strong statement. What about a b2bua doing weird stuff?
> 
> Sure, no protocol mechanism can prevent a B2BUA from doing something.  We will clarify to just say redirection, since that operation is well defined in RFC 3261.
> 
>> 
>> -- REQ-8: "If the UAS does not understand the UUI mechanism, the request will fail."
>> 
>> Based on the routing requirement, shouldn't that say that if the request cannot be routed to a UAS that understands the UUI mechanism, the request will fail?
> 
> Yes, this is clearer.
> 
>> 
>> -- REQ-12: 
>> 
>> What degree of certainty is required here? (i.e. strong identity?) If implied by the SIP dialog, does that impact expectations on what sort of authn must happen at the SIP layer?
> 
> This is not meant to imply strong identity.  And since UUI data can appear in a response, there aren't really any strong methods available with SIP.   The UUI mechanism does not introduce stronger authorization requirements for SIP, but instead the mechanism needs to be able to utilize existing SIP approaches.
> 
>> 
>> -- REQ 13:
>> 
>> I'm not sure I understand how this interacts with the ability for intermediaries to remove UUI. Should this be detectable by the endpoints? Or is that ability limited to the hop-by-hop case, or require no integrity protection?
> 
> Yes, there are tradeoffs between this requirement and requirement REQ-9.  Hop-by-hop protection is one way to resolve this interaction.
> 
>> 
>> Nits/editorial comments:
>> 
>> -- section 4, 2nd paragraph: "The UUI mechanisim should support both of these approaches"
>> 
>> Should that be a numbered requirement? You've got requirements to support e2e and hop-by-hop, but no requirement that mentions SIP layer vs application layer.
> 
> Actually, this sentence is misplaced.  There isn't really a requirement to support both of these.  I'll remove it to avoid confusion.
> 
> 
>