[Stox] Review of -core

Saúl Ibarra Corretgé <saul@ag-projects.com> Tue, 30 July 2013 13:41 UTC

Return-Path: <saul@ag-projects.com>
X-Original-To: stox@ietfa.amsl.com
Delivered-To: stox@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D46A311E8225 for <stox@ietfa.amsl.com>; Tue, 30 Jul 2013 06:41:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.688
X-Spam-Level:
X-Spam-Status: No, score=-1.688 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HELO_MISMATCH_NET=0.611, MIME_8BIT_HEADER=0.3]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id PJ8b3RUgIA23 for <stox@ietfa.amsl.com>; Tue, 30 Jul 2013 06:41:49 -0700 (PDT)
Received: from mail.sipthor.net (node06.dns-hosting.info [85.17.186.6]) by ietfa.amsl.com (Postfix) with ESMTP id 5BB4711E8219 for <stox@ietf.org>; Tue, 30 Jul 2013 06:41:48 -0700 (PDT)
Received: by mail.sipthor.net (Postfix, from userid 5001) id 49033B35DE; Tue, 30 Jul 2013 15:41:44 +0200 (CEST)
Received: from dhcp-152d.meeting.ietf.org (dhcp-152d.meeting.ietf.org [130.129.21.45]) by mail.sipthor.net (Postfix) with ESMTPSA id B0255B017A for <stox@ietf.org>; Tue, 30 Jul 2013 15:41:43 +0200 (CEST)
From: =?iso-8859-1?Q?Sa=FAl_Ibarra_Corretg=E9?= <saul@ag-projects.com>
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: quoted-printable
Message-Id: <B41C24DF-D89B-47B0-98D5-C176AC13EF81@ag-projects.com>
Date: Tue, 30 Jul 2013 15:41:42 +0200
To: "stox@ietf.org" <stox@ietf.org>
Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\))
X-Mailer: Apple Mail (2.1508)
Subject: [Stox] Review of -core
X-BeenThere: stox@ietf.org
X-Mailman-Version: 2.1.12
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: Tue, 30 Jul 2013 13:41:55 -0000

Hi,

As promised, here is my review of the -core document:

- Sec 4.3 defines instant specific mapping, which maps a public GRUU to a full JID and vice versa, but doesn't contain any text on temporary GRUUs. They take the following form: sip:yyd894uue094ur4@1.2.3.4:5060;gr. I don't think we can work with those, since there is nothing we can map to the resource part of the full JID, so I think we need some text to clarify this.

- Sec 5.1 maps the XMPP unexpected-request error to SIP code 491, which is very tied to the dialog semantics IMHO. I tried to find a better matching code but I couldn't though. Maybe a 500 with a reasonable Retry-After header value?

- Sec 5.1 should me map the SIP outbound (RFC5626) error codes? Those are 430, and 439 which could be translated to bad-request.


* Editorial nits:

- Sec 3: "This assumption not meant" -> "This assumption is not meant"


Cheers,

--
Saúl Ibarra Corretgé
AG Projects