Re: [straw] WGLC for draft-ietf-straw-sip-traceroute-01.txt

Hadriel Kaplan <hadriel.kaplan@oracle.com> Mon, 10 March 2014 21:25 UTC

Return-Path: <hadriel.kaplan@oracle.com>
X-Original-To: straw@ietfa.amsl.com
Delivered-To: straw@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 12BCC1A04B0 for <straw@ietfa.amsl.com>; Mon, 10 Mar 2014 14:25:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.748
X-Spam-Level:
X-Spam-Status: No, score=-4.748 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.547, SPF_PASS=-0.001] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id RBwUZsaZTWoM for <straw@ietfa.amsl.com>; Mon, 10 Mar 2014 14:25:34 -0700 (PDT)
Received: from userp1040.oracle.com (userp1040.oracle.com [156.151.31.81]) by ietfa.amsl.com (Postfix) with ESMTP id 09AC01A0329 for <straw@ietf.org>; Mon, 10 Mar 2014 14:25:34 -0700 (PDT)
Received: from ucsinet22.oracle.com (ucsinet22.oracle.com [156.151.31.94]) by userp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with ESMTP id s2ALPR4n014990 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Mon, 10 Mar 2014 21:25:28 GMT
Received: from aserz7021.oracle.com (aserz7021.oracle.com [141.146.126.230]) by ucsinet22.oracle.com (8.14.5+Sun/8.14.5) with ESMTP id s2ALPRYe017490 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Mon, 10 Mar 2014 21:25:27 GMT
Received: from abhmp0013.oracle.com (abhmp0013.oracle.com [141.146.116.19]) by aserz7021.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id s2ALPQiu004515; Mon, 10 Mar 2014 21:25:26 GMT
Received: from [10.0.1.12] (/66.31.4.61) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Mon, 10 Mar 2014 14:25:26 -0700
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\))
From: Hadriel Kaplan <hadriel.kaplan@oracle.com>
In-Reply-To: <4A4F136CBD0E0D44AE1EDE36C4CD9D99739A21CD@VOEXM31W.internal.vodafone.com>
Date: Mon, 10 Mar 2014 17:25:24 -0400
Content-Transfer-Encoding: quoted-printable
Message-Id: <0EE66CBC-EDF2-442B-AA0E-4F4829816DE4@oracle.com>
References: <4A4F136CBD0E0D44AE1EDE36C4CD9D9973905789@VOEXM31W.internal.vodafone.com> <41F7292D-28D8-4192-A4BF-22738BF3939D@oracle.com> <4A4F136CBD0E0D44AE1EDE36C4CD9D9973914B4F@VOEXM31W.internal.vodafone.com> <A43C01B2-7049-4977-9922-C4F78FD411AD@oracle.com> <4A4F136CBD0E0D44AE1EDE36C4CD9D99739A21CD@VOEXM31W.internal.vodafone.com>
To: "Dawes, Peter, Vodafone Group" <Peter.Dawes@vodafone.com>
X-Mailer: Apple Mail (2.1874)
X-Source-IP: ucsinet22.oracle.com [156.151.31.94]
Archived-At: http://mailarchive.ietf.org/arch/msg/straw/WGXH52ulNPociYd8ttgvND-HTMo
Cc: "straw@ietf.org" <straw@ietf.org>
Subject: Re: [straw] WGLC for draft-ietf-straw-sip-traceroute-01.txt
X-BeenThere: straw@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Sip Traversal Required for Applications to Work \(STRAW\) working group discussion list" <straw.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/straw>, <mailto:straw-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/straw/>
List-Post: <mailto:straw@ietf.org>
List-Help: <mailto:straw-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/straw>, <mailto:straw-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 10 Mar 2014 21:25:36 -0000

On Mar 10, 2014, at 3:33 PM, Dawes, Peter, Vodafone Group <Peter.Dawes@vodafone.com> wrote:

> Hi Hadriel,
> Apologies for not being very clear with my comment about lack of clarity :-)
> 
> The draft describes behaviour for a UAC, B2BUA, and UAS. Is the paragraph below from clause 3.1 an instruction to a B2BUA implementer on how to process Max-Forwards or a warning that some B2BUAs will cause the mechanism to fail?

It's an instruction - it has the word "MUST", and it says right at the end:
  ...This document also
  requires such behavior in order to succeed, therefore a B2BUA
  supporting the traceroute mechanism defined in this document MUST
  also comply with [draft-loop-detection].


> A couple of other small questions, is the first sentence "As currently defined in [RFC3261], the UAS half of a B2BUA does not technically need to inspect the Max-Forwards header field value for received requests - only Proxies do. " needed?

Yes, I think it sets the context for this being a change in behavior.


> Also, regarding complying with [draft-loop-detection] does a B2BUA need to do more than "inspect the value in order to prevent loops, as well as copy and decrement the value as if it were a Proxy"? If so it would be good to say something like ", therefore a B2BUA supporting the traceroute mechanism defined in this document MUST also comply with the Max-Forwards behaviour and all other procedures in [draft-loop-detection]"

Isn't that synonymous with "MUST also comply with [draft-loop-detection]"?

I must be missing something obvious to you in reading that paragraph, because it sounds like you're saying it doesn't say X when it says X to me.
:)

-hadriel


> -----Original Message-----
> From: Hadriel Kaplan [mailto:hadriel.kaplan@oracle.com] 
> Sent: 10 March 2014 17:09
> To: Dawes, Peter, Vodafone Group
> Cc: straw@ietf.org
> Subject: Re: [straw] WGLC for draft-ietf-straw-sip-traceroute-01.txt
> 
> Hi Peter,
> sorry for the delay in responding to this!
> comments inline...
> 
> On Dec 17, 2013, at 7:04 AM, Dawes, Peter, Vodafone Group <Peter.Dawes@vodafone.com> wrote:
> 
>> I think the draft text in clause 3.1 Processing a Received Max-Forwards Header Field (lines 206 - 208)] "As currently defined in [RFC3261], the UAS half of a B2BUA does not technically need to inspect the Max-Forwards header field value for received requests - only Proxies do." should be clearer in terms of what the implementer actually has to do. 
> 
> It says:
>   As currently defined in [RFC3261], the UAS half of a B2BUA does not
>   technically need to inspect the Max-Forwards header field value for
>   received requests - only Proxies do.  This behavior was updated by
>   [draft-loop-detection], such that a compliant B2BUA needs to both
>   inspect the value in order to prevent loops, as well as copy and
>   decrement the value as if it were a Proxy.  This document also
>   requires such behavior in order to succeed, therefore a B2BUA
>   supporting the traceroute mechanism defined in this document MUST
>   also comply with [draft-loop-detection].
> 
> 
> How is that not clear?
> 
> 
>> You replied that "...I think it is in fact in doubt.  I know of many B2BUAs which do in fact decrement max-forwards. (and I think they're right to)". So what does the implementer do? Be happy that they might not need to change too much? Make sure all of their B2BUAs decrement Max-Forwards whether they will loopback media or not? Be aware that if they don't want their B2BUA to decrement Max-Forwards then they can't use this mechanism? Something else?
> 
> See above.
> 
> -hadriel
>