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

"Dawes, Peter, Vodafone Group" <Peter.Dawes@vodafone.com> Tue, 11 March 2014 09:26 UTC

Return-Path: <Peter.Dawes@vodafone.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 D7EB51A03E8 for <straw@ietfa.amsl.com>; Tue, 11 Mar 2014 02:26:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.447
X-Spam-Level:
X-Spam-Status: No, score=-2.447 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RP_MATCHES_RCVD=-0.547] 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 GEnzZOgD8a8t for <straw@ietfa.amsl.com>; Tue, 11 Mar 2014 02:26:12 -0700 (PDT)
Received: from mailout08.vodafone.com (mailout08.vodafone.com [195.232.224.77]) by ietfa.amsl.com (Postfix) with ESMTP id 8FD5E1A03E2 for <straw@ietf.org>; Tue, 11 Mar 2014 02:26:12 -0700 (PDT)
Received: from mailint08.vodafone.com (localhost [127.0.0.1]) by mailout08.vodafone.com (Postfix) with ESMTP id 925B7221E24 for <straw@ietf.org>; Tue, 11 Mar 2014 10:26:05 +0100 (CET)
Received: from VOEXC01W.internal.vodafone.com (voexc01w.dc-ratingen.de [145.230.101.21]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by mailint08.vodafone.com (Postfix) with ESMTPS id 836B5221DC4; Tue, 11 Mar 2014 10:26:05 +0100 (CET)
Received: from VOEXC17W.internal.vodafone.com (145.230.101.19) by VOEXC01W.internal.vodafone.com (145.230.101.21) with Microsoft SMTP Server (TLS) id 14.3.146.2; Tue, 11 Mar 2014 10:26:05 +0100
Received: from VOEXM31W.internal.vodafone.com ([169.254.7.57]) by voexc17w.internal.vodafone.com ([145.230.101.19]) with mapi id 14.03.0146.002; Tue, 11 Mar 2014 10:26:04 +0100
From: "Dawes, Peter, Vodafone Group" <Peter.Dawes@vodafone.com>
To: Hadriel Kaplan <hadriel.kaplan@oracle.com>
Thread-Topic: [straw] WGLC for draft-ietf-straw-sip-traceroute-01.txt
Thread-Index: AQHO+pSfJfgZsqbMXE+cNvCMDQ15NppYOTBAgILHwQCAADEgIIAAFoEAgADWa/A=
Date: Tue, 11 Mar 2014 09:26:04 +0000
Message-ID: <4A4F136CBD0E0D44AE1EDE36C4CD9D99739A23A8@VOEXM31W.internal.vodafone.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> <0EE66CBC-EDF2-442B-AA0E-4F4829816DE4@oracle.com>
In-Reply-To: <0EE66CBC-EDF2-442B-AA0E-4F4829816DE4@oracle.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: http://mailarchive.ietf.org/arch/msg/straw/-0OH1j_nzJaCgRgoIWodrrNbM24
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: Tue, 11 Mar 2014 09:26:15 -0000

Hi Hadriel,
My confusion with 3.1 came from the mixture of introductory description (the UAS half of a B2BUA does not technically need to inspect the Max-Forwards header) with normative specification in the same paragraph, but I accept that the MUST makes it clear which is which. The other comments related to potentially shortening the text a little and not to errors or ambiguity.

Thanks for the answers, I have no more comments.

Peter

-----Original Message-----
From: Hadriel Kaplan [mailto:hadriel.kaplan@oracle.com] 
Sent: 10 March 2014 21:25
To: Dawes, Peter, Vodafone Group
Cc: straw@ietf.org
Subject: Re: [straw] WGLC for draft-ietf-straw-sip-traceroute-01.txt


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
>