[Stox] Review for stox-7248bis-02

"Isomaki Markus (Nokia-TECH/Espoo)" <markus.isomaki@nokia.com> Thu, 16 July 2015 14:18 UTC

Return-Path: <prvs=632e7278f=markus.isomaki@nokia.com>
X-Original-To: stox@ietfa.amsl.com
Delivered-To: stox@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BE8911B3C73 for <stox@ietfa.amsl.com>; Thu, 16 Jul 2015 07:18:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, 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 yg55uW-72vxU for <stox@ietfa.amsl.com>; Thu, 16 Jul 2015 07:18:45 -0700 (PDT)
Received: from nok-msg-2.service.capgemini.fi (nok-msg-2.service.capgemini.fi [145.247.12.203]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 83E2F1B3C67 for <stox@ietf.org>; Thu, 16 Jul 2015 07:18:44 -0700 (PDT)
Received: from unknown (HELO NOKWDCFIEXCH01P.nnok.nokia.com) ([10.50.38.49]) by noi-msg-2.service.capgemini.fi with ESMTP; 16 Jul 2015 17:18:42 +0300
Received: from NOKWDCFIEXCH02P.nnok.nokia.com (10.50.38.50) by NOKWDCFIEXCH01P.nnok.nokia.com (10.50.38.49) with Microsoft SMTP Server (TLS) id 15.0.1044.25; Thu, 16 Jul 2015 17:18:41 +0300
Received: from NOKWDCFIEXCH02P.nnok.nokia.com ([fe80::99d1:400a:d939:3ebe]) by NOKWDCFIEXCH02P.nnok.nokia.com ([fe80::99d1:400a:d939:3ebe%17]) with mapi id 15.00.1044.021; Thu, 16 Jul 2015 17:18:41 +0300
From: "Isomaki Markus (Nokia-TECH/Espoo)" <markus.isomaki@nokia.com>
To: "stox@ietf.org" <stox@ietf.org>
Thread-Topic: Review for stox-7248bis-02
Thread-Index: AdC/0cUiLOk2SX8aQIy0VVTyZI/MLQ==
Date: Thu, 16 Jul 2015 14:18:40 +0000
Message-ID: <9315d702a9534242b9c36b4b93e19a45@NOKWDCFIEXCH02P.nnok.nokia.com>
Accept-Language: en-US, fi-FI
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.50.32.5]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <http://mailarchive.ietf.org/arch/msg/stox/KEtkmxk-9WtNGEOXxsjv5uaIj_8>
Subject: [Stox] Review for stox-7248bis-02
X-BeenThere: stox@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: SIP-TO-XMPP Working Group discussion list <stox.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/stox>, <mailto:stox-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/stox/>
List-Post: <mailto:stox@ietf.org>
List-Help: <mailto:stox-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/stox>, <mailto:stox-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Jul 2015 14:18:47 -0000

Hi Peter, all,

I did a review for the 7248bis-02 draft and unless I'm missing something, there seem to be still a few confusing things  about the message flow diagrams and the terminology about SIP-to-XMPP and XMPP-to-SIP gateways. (The reason to go for 7248bis was indeed to fix the message flows so they are really the main part of it.)

A minor issue:
- In the diagrams abbreviations S2X GW and X2S GW are used. While most readers can guess what they mean, it would be good to open them explicitly, perhaps in Section 3, Terminology. 

Bigger issues:
- The diagram in Section 4.2.1: It looks weird that the SIP server already generates the (F7) 200 OK message, while the (F8) NOTIFY does not get any response. Shouldn't it be that the 200 OK is generated by the "X2S GW" and then passed forward by the SIP server? 
- The text in 4.2.1 is somewhat confusing as well, as it first (understandably) talks about XMPP-to-SIP GW as shown in the diagram ("S2X"), but then (after Example 2) talks a few times about SIP-to-XMPP GW as well. I believe this is coming from 7248, which does have the two GWs as explicit entities in the diagram. But in 7248bis they are merged, and the current text makes the reader wonder if SIP-to-XMPP GW refers to the same entity as XMPP-to-SIP GW as shown, or whether there is another implicit/invisible GW function included. 
- The same issue occurs in 4.3.1 in reverse. There, the diagram only has "S2X GW" but the text contains both SIP-to-XMPP and XMPP-to-SIP. 

So we have gotten rid of the double gateways of 7248 in the diagrams, but they still somehow loom in the text and terminology. I think there are a couple of options to clarify this: Either always refer to the gateway as XMPP-to-SIP (X2S) in those sections that are about XMPP-to-SIP, and vice versa. Or, make the gateway direction neutral and just call it something like SIP-XMPP GW everywhere. Or, explain some more about the directions.

Sorry to be late about this, but I guess this is the last chance and we should definitely get the gateways right this time :-O

Markus