Re: [Insipid] Pre-WGLC INSIPID Session-ID Review Comments

"Paul E. Jones" <paulej@packetizer.com> Fri, 05 September 2014 01:00 UTC

Return-Path: <paulej@packetizer.com>
X-Original-To: insipid@ietfa.amsl.com
Delivered-To: insipid@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 47D431A02E3 for <insipid@ietfa.amsl.com>; Thu, 4 Sep 2014 18:00:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.66
X-Spam-Level:
X-Spam-Status: No, score=-1.66 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_ADSP_ALL=0.8, DKIM_SIGNED=0.1, RP_MATCHES_RCVD=-0.668, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, T_DKIM_INVALID=0.01] autolearn=no
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 Asu7X3E6Qma0 for <insipid@ietfa.amsl.com>; Thu, 4 Sep 2014 18:00:30 -0700 (PDT)
Received: from dublin.packetizer.com (dublin.packetizer.com [75.101.130.125]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1C0311A02D9 for <insipid@ietf.org>; Thu, 4 Sep 2014 18:00:29 -0700 (PDT)
Received: from [192.168.1.20] (cpe-024-211-197-136.nc.res.rr.com [24.211.197.136]) (authenticated bits=0) by dublin.packetizer.com (8.14.5/8.14.5) with ESMTP id s8510Kv2021400 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Thu, 4 Sep 2014 21:00:22 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=packetizer.com; s=dublin; t=1409878822; bh=ATWMUIkjUkVwGQ+k7wk89dsHMjZPx4a+Had4Rgztb+4=; h=From:To:Subject:Cc:Date:Message-Id:In-Reply-To:Reply-To: Mime-Version:Content-Type:Content-Transfer-Encoding; b=oSoXagV2Xh+hPGamkUyH/LmgUHd+3LunUg0X8jJOFJRCvesA1mlRoG4isvpsMpJUt jKDY8l0uFaG5ciq+AxZMRbKSEKr8WYBLNgWehUaja+6YtYMkxuMPqcjjAX6L8JNfmq DM1gpNud9hJ6D/3I4onKPo30oYh+/5YzJwnXbgd0=
From: "Paul E. Jones" <paulej@packetizer.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>, "Adam Gensler (agensler)" <agensler@cisco.com>
Date: Fri, 05 Sep 2014 01:00:55 +0000
Message-Id: <emc4334dac-8688-4c8a-9018-f9cf5ea0130d@sydney>
In-Reply-To: <7594FB04B1934943A5C02806D1A2204B1D4396A4@ESESSMB209.ericsson.se>
User-Agent: eM_Client/6.0.20617.0
Mime-Version: 1.0
Content-Type: text/plain; format="flowed"; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: http://mailarchive.ietf.org/arch/msg/insipid/4o9aZs4_2KBnHXw6V9YwzFbcjNI
Cc: "insipid@ietf.org" <insipid@ietf.org>
Subject: Re: [Insipid] Pre-WGLC INSIPID Session-ID Review Comments
X-BeenThere: insipid@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
Reply-To: "Paul E. Jones" <paulej@packetizer.com>
List-Id: SIP Session-ID discussion list <insipid.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/insipid>, <mailto:insipid-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/insipid/>
List-Post: <mailto:insipid@ietf.org>
List-Help: <mailto:insipid-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/insipid>, <mailto:insipid-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Sep 2014 01:00:32 -0000

Christer,

Thanks for the feedback.  I'll try to address each point:


>General:
>-----------
>
>- The document claims that the Session-ID is true end-to-end, because 
>it will not be modified by intermediaries. But, there is no text on how 
>that is achieved. Obviously, we cannot guarantee that it won't happen 
>(even if there is "MUST NOT" wording), but I think you need to have 
>some text which talks about the reasons why Call-IDs are modified, and 
>why those reasons don't apply to the Session-ID.

In the introduction, we specify two examples of why the current Call-ID 
values are not suitable, including the fact that SBCs change the values 
as they go through.  We also mention protocol interworking as an issue.

I'm not sure what we can write into the document to ensure that devices 
do not change the Session-ID.  It's certainly possible, just as the 
Call-ID is changed.  However, changing the session ID would mean the 
device is not compliant with the specification.  That could happen on 
purpose (violating the spec) or it could happen because there is an 
older device that does not know the header and simply discards it.  
That's certainly true of any new protocol element one might introduce.  
If you think we should say something explicit about this, where should 
we say it and how should we phrase it?


>- The document talks about interoperability with H.323, but nowhere is 
>there any text showing that both protocols now use the same format for 
>the session id.

That is true, though RFC 7602 (requirements) mentions that a parallel 
effort is taking place in SG16.  In fact, here's the draft text: 
http://ftp3.itu.int/av-arch/avc-site/2013-2016/1406_Sap/Draft-H.460.SessionID.zip.

That text was accepted as the baseline text for an extension to H.323.  
However, it's not something we can reference right now.  Do you feel we 
should re-state the fact that the ITU is undertaking the same work for 
H.323, or do you think leaving that fact in the requirements document is 
enough?

>Section 5:
>-------------
>
>I suggest to re-name the section name to "SIP Session-ID Header Field"
>
>You need to split the text into subsections.
>
>Something like:
>
>5.1 General
>5.2 Syntax
>5.3 UA Behavior
>5.3.1 UAC Behavior
>5.3.2 UAS Behavior
>5.4 Proxy Behavior
>5.5 B2BUA Behavior

I think renaming this is a good idea.  Given the whole document is about 
SIP, I think we can get rid of that, too.  I think "The Session-ID 
Header Field" would be sufficient.

However, I think making the structural changes is probably unnecessary.  
Splitting UAS and UAC behavior, for example, will result in a lot of 
redundant text.   The same is said of a SIP proxy or B2BUA.  Both are 
intermediaries and should operate in the same way. We considered that 
arrangement before, but felt it was simpler to just describe endpoint 
behavior and intermediary behavior as we have it.

>Section 6:
>-------------
>
>The text in this section should go into UA Behavior.
>
>Section 7:
>-------------
>
>The text in this section should go into Proxy Behavior and B2BUA 
>Behavior.
>
>
>I do have more comments on section 6 and 7, but I'd like to see the 
>restructuring (as suggested in my comment on section 5) before giving 
>those, because some may go away.
OK, let's talk about those.  I just submitted -08 with a few minor 
changes.

Paul