Re: [Stox] Review on -im

Saúl Ibarra Corretgé <saul@ag-projects.com> Tue, 30 July 2013 15:54 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 BA6BC21F8546 for <stox@ietfa.amsl.com>; Tue, 30 Jul 2013 08:54:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.746
X-Spam-Level:
X-Spam-Status: No, score=-0.746 tagged_above=-999 required=5 tests=[AWL=-0.528, BAYES_00=-2.599, HELO_MISMATCH_NET=0.611, J_CHICKENPOX_42=0.6, MIME_8BIT_HEADER=0.3, SARE_MLH_Stock1=0.87]
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 SZDkb1bDGjYw for <stox@ietfa.amsl.com>; Tue, 30 Jul 2013 08:54:38 -0700 (PDT)
Received: from mail.sipthor.net (node06.dns-hosting.info [85.17.186.6]) by ietfa.amsl.com (Postfix) with ESMTP id A0A3F21F9A35 for <stox@ietf.org>; Tue, 30 Jul 2013 08:54:34 -0700 (PDT)
Received: by mail.sipthor.net (Postfix, from userid 5001) id 7A6ECB35DB; Tue, 30 Jul 2013 17:54:27 +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 70CCAB017C for <stox@ietf.org>; Tue, 30 Jul 2013 17:54:26 +0200 (CEST)
Content-Type: text/plain; charset="iso-8859-1"
Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\))
From: Saúl Ibarra Corretgé <saul@ag-projects.com>
In-Reply-To: <7AE79F98-B222-4BBC-BC02-1606AF8F34A9@ag-projects.com>
Date: Tue, 30 Jul 2013 17:54:25 +0200
Content-Transfer-Encoding: quoted-printable
Message-Id: <08B37233-F92B-49C9-AC75-6A82B9FDBF21@ag-projects.com>
References: <7AE79F98-B222-4BBC-BC02-1606AF8F34A9@ag-projects.com>
To: "stox@ietf.org" <stox@ietf.org>
X-Mailer: Apple Mail (2.1508)
Subject: Re: [Stox] Review on -im
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 15:54:44 -0000

One more thing: IMHO we should also mention that "headline" XMPP messages should be translated like the normal ones in this spec. Not in -chat though.

On Jul 30, 2013, at 4:19 PM, Saúl Ibarra Corretgé <saul@ag-projects.com> wrote:

> Hi,
> 
> Here is my review on the -im document:
> 
> - Section 3 suggests that the thread element in XMPP should be mapped to the Call-ID header. I see why this was done, but it looks a bit like abusing it. I would probably take it out or define it as a CPIM header, in case CPIM is used. Thoughts?
> 
> - Section 5 only mentions text/plain and text/html, I think we should add message/cpim in there.
> 
> - Similarly to chat, I think we should define a mapping between XMPP chatstates (XEP-0085) and the iscomosing indication (RFC 3994), but in this case there would be no mapping for the 'gone' state.
> 
> 
> * Editorial nits:
> 
> - In the examples: From, To and Contact header URIs are not enclosed in angle brackets
> - In the example in section 4, the RURI should have the gr parameter as well,as the To header is not used for routing
> 
> 
> Cheers,
> 
> --
> Saúl Ibarra Corretgé
> AG Projects
> 
> 
> 
> _______________________________________________
> stox mailing list
> stox@ietf.org
> https://www.ietf.org/mailman/listinfo/stox

--
Saúl Ibarra Corretgé
AG Projects