Re: [Stox] Extended WGLC for draft-ietf-stox-groupchat-04 [was: WGLC for draft-ietf-stox-groupchat-04]

Matt Ryan <mryan@getjive.com> Fri, 30 May 2014 20:38 UTC

Return-Path: <mryan@getjive.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 D5CE21A8788 for <stox@ietfa.amsl.com>; Fri, 30 May 2014 13:38:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, 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 SANAlvEkUw1P for <stox@ietfa.amsl.com>; Fri, 30 May 2014 13:38:26 -0700 (PDT)
Received: from mail-ig0-x234.google.com (mail-ig0-x234.google.com [IPv6:2607:f8b0:4001:c05::234]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D0E791A6FE6 for <stox@ietf.org>; Fri, 30 May 2014 13:38:25 -0700 (PDT)
Received: by mail-ig0-f180.google.com with SMTP id c1so1316611igq.7 for <stox@ietf.org>; Fri, 30 May 2014 13:38:21 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=getjive.com; s=mail; h=message-id:date:from:organization:user-agent:mime-version:to :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=oLSruS1FL04M5cfHFe750ftTUBgFaILKXELLSi+COYo=; b=FFh9I2fYvXuW5F2HvBAHoBF+QtwnuNodQ46g+BTywMzscVQKuI4VQUiLlgG28/abzj VFQ0ahpGerCWXljDKXxl9VEUFd/bYgxHEZHshgHdFV6+04tPjmnaDcaiF9Nvj/mNZroR vIaJ+l9f9oxGub0lpSt1XhAsirC7qjgRcB1bc=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:organization:user-agent :mime-version:to:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=oLSruS1FL04M5cfHFe750ftTUBgFaILKXELLSi+COYo=; b=XGKLnKOdYs0ToCH4eN0pivDwq22WRcv0asOoRSKYq/V4+LWm24a29BF6LeCGZUz01N tOzIpCcEtperCvUqxj6K4mqYSdMD4Vxmw//oMy+5JQFVry1j+4p10HC1VHaBpEeXi0vl Qhgv8HAhIm9P6RTpQkOGs3vlgo4gcTXhDicpJeb59nZqEwroDeWPozrEkDdH0dXcCDJ7 ZnRVphgTJKBiiQe2lp3lVKxCW0EbpHkxdJttSbgInp3SumdmC7ArhLIdLCMy1a1AFAr5 XZP85b9D3LdcnjddDc7t0G6DsHGp9V6Ko8A7LtnbdCBqWd0uQ8PvSd3MTeMvKdCORQ4+ UV3A==
X-Gm-Message-State: ALoCoQm6VcRvLmFqip2HfG5fjPt0AV8x5cVToX1lqCIMGFrc7NwueFinTD6IoRF3fqC8bmR1eJhx
X-Received: by 10.50.254.132 with SMTP id ai4mr9148469igd.30.1401482300974; Fri, 30 May 2014 13:38:20 -0700 (PDT)
Received: from [10.20.200.236] ([199.87.120.126]) by mx.google.com with ESMTPSA id h5sm7971179igi.4.2014.05.30.13.38.19 for <stox@ietf.org> (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 30 May 2014 13:38:20 -0700 (PDT)
Message-ID: <5388EC3B.70405@getjive.com>
Date: Fri, 30 May 2014 14:38:19 -0600
From: Matt Ryan <mryan@getjive.com>
Organization: Jive Communications, Inc.
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:24.0) Gecko/20100101 Thunderbird/24.5.0
MIME-Version: 1.0
To: stox@ietf.org
References: <0E1BD27E-E0B5-4B61-8451-5B3ED8B649A2@jitsi.org> <DA536B73-D11A-40AD-901B-1428BC7376E1@jitsi.org> <5C094162-79EC-47AA-8861-1DC2F19E96E8@nostrum.com> <53852400.2070704@stpeter.im>
In-Reply-To: <53852400.2070704@stpeter.im>
X-Enigmail-Version: 1.6
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/stox/7FM5mKMzhSK18kjK9Im2lmjQXIs
Subject: Re: [Stox] Extended WGLC for draft-ietf-stox-groupchat-04 [was: WGLC for draft-ietf-stox-groupchat-04]
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: <http://www.ietf.org/mail-archive/web/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: Fri, 30 May 2014 20:38:29 -0000

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Comments below.


On 5/27/14, 5:47 PM, Peter Saint-Andre wrote:
> Hi Ben, thanks for the feedback! Comments inline.
> 
> On 5/2/14, 2:44 PM, Ben Campbell wrote:
>> Hi,
>> 
>> Here's some belated comments:
>> 
(snip)
>> I see some issues with the SIP interaction. It may be that I am
>> confused by the two-gateway model, but you have a number of
>> transactions where the SIP responses do not go back to the same
>> entity that sent the SIP request. I suspect the intent was that
>> one gateway or the other would handle things depending on which
>> side initiated communication, but you've got dialogs (even
>> individual transactions) split across them. If this is intended,
>> then they would need to share SIP dialog and transaction state
>> somehow.
> 
> Hmmmmmm.
> 
> You make a good point. The diagrams here are not consistent with
> the architectural assumptions we described in the stox-core spec
> (now RFC 7247)...
> 
> ######################################################### #
> # #                 +-----+                               # #
> | S2X |                               # #   +-------------+ GW
> |<...........>+-------------+   # #   | SIP Server  +-----+
> | XMPP Server |   # #   | example.net |             +-----+
> example.com |   # #   +-------------+<***********>| X2S
> +-------------+   # #         *                     | GW  |  :
> # #         *                     +-----+  :              # #
> *                              :              # #
> romeo@example.net             juliet@example.com   # #
> # #########################################################
> 
> Legend: XMPP = ... or : SIP = *
> 
> Figure 1: Possible Gateway Deployment Architecture
> 
> Note that bidirectional communication between the SIP server and
> the XMPP server can be established over either SIP or XMPP.  If the
> XMPP user initiates the interaction, then the XMPP server would
> invoke its XMPP-to-SIP gateway; thus, the communication would occur
> over SIP. If the SIP user initiates the interaction, then the SIP
> server would invoke its SIP-to-XMPP gateway; thus, the
> communication would occur over XMPP.
> 
> That is, the MSRP-to-XMPP gateway doesn't belong as an entity in
> the protocol flow for XMPP MUC to MSRP Multi-party Messaging
> Session (Section 4), and the XMPP-to-MSRP gateway doesn't belong in
> the protocol flow for MSRP Multi-party Messaging Session to XMPP
> MUC (Section 5).If we correct this error, I think most of the
> feedback you provide below can be easily addressed.


This is one of the concerns I also raised in my review of
draft-ietf-stox-chat-06 on 5/12/14, and why I raised my subsequent
question about communication between STOX-capable and non-STOX-capable
entities.

The situation is actually a bit worse:  If the XMPP-to-SIP gateway
sends a SIP invite to the SIP server (or MSRP conference in the
groupchat case) on the receiving end, the SIP session will not get
established until the XMPP-to-SIP gateway receives a 200 OK from the
SIP server, and it will continue to retry this INVITE until the
session is established.

Keeping state on the XMPP side might help (other than you still have
the unnecessary retransmits), but as Peter pointed out, simply
removing the MSRP-to-XMPP gateway in this protocol flow would then
result in the 200 OK returning to the XMPP-to-SIP gateway, which would
complete the establishment of the session also.


- -- 
Matt Ryan
Code Slinger  |  Jive Communications, Inc.
Jive.com  |  mryan@getjive.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.22 (Darwin)
Comment: GPGTools - https://gpgtools.org
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBCgAGBQJTiOw5AAoJEPyFiw1u4O0c/mIIAKDTV4SAwORIuL3GwfqfA2Vg
AKeGMgmKT9RTLhXnh3YjJxetN1XJ9spGcaVUIv4xvlDnAdqqeURiR1jc8uSkLtl5
YPdzERgPrL4tLcum2+/0cVneRvgLqHfwj9O88im9GRtuMTZ8Ny6Vb0+rHHylq7Y+
zKo88Zi3myT7MNXZioa7KTxt+xtTDQz2LxhCkuLeK+Sr/dkG+aQOjI4ZYsPSE7Ne
aatqCXqA3gUbWVe1YmUhCzkSjVTun6OWRkxDvxG9QLIQkFRZnu96FPSrJETYYYwf
QXAyX14yr9beq6NUxUDdsCZJ0URL/mHeuSVe0isG+MhBXHwfNIjEebslh3BoACY=
=bt4O
-----END PGP SIGNATURE-----