Re: [Stox] Reminder WGLC on -im and -chat

Peter Saint-Andre <stpeter@stpeter.im> Mon, 09 December 2013 17:22 UTC

Return-Path: <stpeter@stpeter.im>
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 B4F971AE287 for <stox@ietfa.amsl.com>; Mon, 9 Dec 2013 09:22:16 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.903
X-Spam-Level:
X-Spam-Status: No, score=-1.903 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.001, SPF_HELO_PASS=-0.001, 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 PCc6U_YE1uSZ for <stox@ietfa.amsl.com>; Mon, 9 Dec 2013 09:22:12 -0800 (PST)
Received: from stpeter.im (mailhost.stpeter.im [207.210.219.225]) by ietfa.amsl.com (Postfix) with ESMTP id 4CDC21AE005 for <stox@ietf.org>; Mon, 9 Dec 2013 09:22:12 -0800 (PST)
Received: from ergon.local (unknown [72.163.0.129]) (Authenticated sender: stpeter) by stpeter.im (Postfix) with ESMTPSA id 1A6364010C; Mon, 9 Dec 2013 10:22:07 -0700 (MST)
Message-ID: <52A5FC3E.4070704@stpeter.im>
Date: Mon, 09 Dec 2013 10:22:06 -0700
From: Peter Saint-Andre <stpeter@stpeter.im>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:24.0) Gecko/20100101 Thunderbird/24.1.1
MIME-Version: 1.0
To: Dave Cridland <dave@cridland.net>, Yana Stamcheva <yana@jitsi.org>
References: <648368DC-1BD5-4D6A-B4E0-3D72B2EB3A6D@jitsi.org> <CAKHUCzwRBEDH3_zc6ESz7YYP0FAbKuZEqX3Dc+_bqV_X=fgXGA@mail.gmail.com>
In-Reply-To: <CAKHUCzwRBEDH3_zc6ESz7YYP0FAbKuZEqX3Dc+_bqV_X=fgXGA@mail.gmail.com>
X-Enigmail-Version: 1.6
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
Cc: stox@ietf.org
Subject: Re: [Stox] Reminder WGLC on -im and -chat
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: Mon, 09 Dec 2013 17:22:17 -0000

On 12/9/13 7:22 AM, Dave Cridland wrote:
> World's shortest reviews to try and sneak past the deadline:

Hi Dave, thanks for the review. Comments inline.

> On Wed, Dec 4, 2013 at 1:17 PM, Yana Stamcheva <yana@jitsi.org
> <mailto:yana@jitsi.org>> wrote:
> 
>     draft-ietf-stox-im-05 : http://tools.ietf.org/html/draft-ietf-stox-im-05
>     <http://tools.ietf.org/html/draft-ietf-stox-im-05
>     draft-ietf-stox-chat-03>
> 
> 
> FWIW, I don't think you could actually do much of this wrong - that is,
> beyond -core (and therefore address and error mappings), I suspect
> everything else can fall into a quality-of-implementation issue - so as
> such, I think this only needs to supply guidance and wisdom rather than
> laying down the law - I'm reviewing accordingly.
> 
> This largely seems OK.
> 
> §6 appears to have interesting mangling of the characters. I think
> &amp;#xC3A9; is intended to be &#xC3A9; isn't it?

Yes. <![CDATA[]]> is your friend for escaping purposes.

> I think this is an unfortunate choice of notation, 

It's driven by the limitations of the RFC format.

> actually, because in
> the XMPP case this is valid as-is, whereas in the SIP case it needs to
> be unencoded UTF-8 

What is "unencoded UTF-8"? UTF-8 is itself an encoding. Plus, RFC 3261 says:

   SIP is a text-based protocol and uses the UTF-8 charset (RFC 2279
   [7]).

> - I don't know whether the Content-Type should also
> include a charset parameter for SIP.

Like this?

OLD
Content-Type: text/plain

NEW
Content-Type: text/plain; charset=utf-8

Given that SIP natively "uses the UTF-8 charset", it strikes me that the
charset parameter is unnecessary here.

> It may be better to show either C-style or URL escaping of the UTF-8 in
> the SIP version, or else a <U+C389>, or something. 

I'm not convinced that this is needed.

> Security Considerations should probably refer back to -core as well,
> shouldn't they?

Yes.

>     draft-ietf-stox-chat-03
>     <http://tools.ietf.org/html/draft-ietf-stox-im-05
>     draft-ietf-stox-chat-03> :
>     http://tools.ietf.org/html/draft-ietf-stox-chat-03
> 
> 
> This draft appears to consist mostly of two giant examples. :-)
> 
> However, it seems to be sufficient guidance to build an interoperable
> gateway, so I'm happy this is ready - however as with the above,
> Security Considerations should probably refer back to -im and -core.

Agreed.

Peter

-- 
Peter Saint-Andre
https://stpeter.im/