Re: [xmpp] Fwd: I-D Action: draft-ietf-xmpp-websocket-04.txt

Florian Zeitz <florob@babelmonkeys.de> Mon, 21 April 2014 00:10 UTC

Return-Path: <florob@babelmonkeys.de>
X-Original-To: xmpp@ietfa.amsl.com
Delivered-To: xmpp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A12861A00CD for <xmpp@ietfa.amsl.com>; Sun, 20 Apr 2014 17:10:41 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.822
X-Spam-Level:
X-Spam-Status: No, score=-1.822 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_EQ_DE=0.35, RP_MATCHES_RCVD=-0.272] 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 0UZNBZydTCAF for <xmpp@ietfa.amsl.com>; Sun, 20 Apr 2014 17:10:37 -0700 (PDT)
Received: from babelmonkeys.de (babelmonkeys.de [IPv6:2a02:d40:3:1:10a1:5eff:fe52:509]) by ietfa.amsl.com (Postfix) with ESMTP id 5AAB81A0073 for <xmpp@ietf.org>; Sun, 20 Apr 2014 17:10:37 -0700 (PDT)
Received: from xdsl-78-34-253-121.netcologne.de ([78.34.253.121] helo=[192.168.0.140]) by babelmonkeys.de with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from <florob@babelmonkeys.de>) id 1Wc1of-0005e2-Ld for xmpp@ietf.org; Mon, 21 Apr 2014 02:10:33 +0200
Message-ID: <535461F1.60901@babelmonkeys.de>
Date: Mon, 21 Apr 2014 02:10:25 +0200
From: Florian Zeitz <florob@babelmonkeys.de>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0
MIME-Version: 1.0
To: xmpp@ietf.org
References: <20140420234508.15834.69854.idtracker@ietfa.amsl.com> <70B56605-EC89-4DA4-86B5-689158EF467B@andyet.net>
In-Reply-To: <70B56605-EC89-4DA4-86B5-689158EF467B@andyet.net>
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/xmpp/zGMUC2-GyPKZ8ynvIG3oAUxdSaU
Subject: Re: [xmpp] Fwd: I-D Action: draft-ietf-xmpp-websocket-04.txt
X-BeenThere: xmpp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: XMPP Working Group <xmpp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/xmpp>, <mailto:xmpp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/xmpp/>
List-Post: <mailto:xmpp@ietf.org>
List-Help: <mailto:xmpp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/xmpp>, <mailto:xmpp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 21 Apr 2014 00:10:42 -0000

On 21.04.2014 01:52, Lance Stout wrote:
> FYI
> 
> This version clarifies the stream initiation exchange by giving an example and explicitly stating 
> that the attributes of the <open /> element match what is prescribed by Section 4.7 of RFC 6120.
> 
Having read the -02 to -03, and -03 to -04 diffs, I am quite content
with the current version. I feel all feedback has been adequately addressed.

One last remark I have is that I dislike (the formatting of) the example
in Section 3.3.3:

"
 -- WS Message boundary --	
 <stream:features xmlns:stream="http://etherx.jabber.org/streams">	
   <bind xmlns="urn:ietf:params:xml:ns:xmpp-bind"/>	
 </stream:features>	
 -- WS Message boundary --	
 <error xmlns="http://etherx.jabber.org/streams">	
   <host-unknown xmlns='urn:ietf:params:xml:ns:xmpp-streams'/>	
 </error>	
 -- WS Message boundary --	
 <message xmlns="jabber:client" xml:lang="en">	
   <body>Every WebSocket message is parsable by itself.</body>	
 </message>
"

It makes it look like this is a valid protocol flow, which it clearly is
not.
When first reading it, I was stuck pondering why a <message/> would be
transmitted after a stream error (and a host-unknown stream error after
features), until I realised this were really 3 separate examples.

Other than that, thank you for your great work Lance.

Best regards,
Florian Zeitz

> 
> 
> Begin forwarded message:
> 
>> From: internet-drafts@ietf.org
>> Subject: [xmpp] I-D Action: draft-ietf-xmpp-websocket-04.txt
>> Date: April 20, 2014 at 4:45:08 PM PDT
>> To: i-d-announce@ietf.org
>> Cc: xmpp@ietf.org
>>
>>
>> A New Internet-Draft is available from the on-line Internet-Drafts directories.
>> This draft is a work item of the Extensible Messaging and Presence Protocol Working Group of the IETF.
>>
>>        Title           : An XMPP Sub-protocol for WebSocket
>>        Authors         : Lance Stout
>>                          Jack Moffitt
>>                          Eric Cestari
>> 	Filename        : draft-ietf-xmpp-websocket-04.txt
>> 	Pages           : 14
>> 	Date            : 2014-04-20
>>
>> Abstract:
>>   This document defines a binding for the XMPP protocol over a
>>   WebSocket transport layer.  A WebSocket binding for XMPP provides
>>   higher performance than the current HTTP binding for XMPP.
>>
>>
>> The IETF datatracker status page for this draft is:
>> https://datatracker.ietf.org/doc/draft-ietf-xmpp-websocket/
>>
>> There's also a htmlized version available at:
>> http://tools.ietf.org/html/draft-ietf-xmpp-websocket-04
>>
>> A diff from the previous version is available at:
>> http://www.ietf.org/rfcdiff?url2=draft-ietf-xmpp-websocket-04
>>
>>
>> Please note that it may take a couple of minutes from the time of submission
>> until the htmlized version and diff are available at tools.ietf.org.
>>
>> Internet-Drafts are also available by anonymous FTP at:
>> ftp://ftp.ietf.org/internet-drafts/
>>
>> _______________________________________________