Re: [xmpp] [Gen-art] Gen-ART review for draft-ietf-xmpp-websocket-07
Peter Saint-Andre <stpeter@stpeter.im> Tue, 08 July 2014 15:22 UTC
Return-Path: <stpeter@stpeter.im>
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 DECA61B2B17; Tue, 8 Jul 2014 08:22:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.553
X-Spam-Level:
X-Spam-Status: No, score=-2.553 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.651, 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 m2J5t4JiSB_0; Tue, 8 Jul 2014 08:22:47 -0700 (PDT)
Received: from stpeter.im (mailhost.stpeter.im [207.210.219.225]) by ietfa.amsl.com (Postfix) with ESMTP id 13BCA1B2B16; Tue, 8 Jul 2014 08:22:47 -0700 (PDT)
Received: from aither.local (unknown [24.8.184.175]) (Authenticated sender: stpeter) by stpeter.im (Postfix) with ESMTPSA id 9D35440E38; Tue, 8 Jul 2014 09:22:42 -0600 (MDT)
Message-ID: <53BC0CC1.6020409@stpeter.im>
Date: Tue, 08 Jul 2014 09:22:41 -0600
From: Peter Saint-Andre <stpeter@stpeter.im>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: "Romascanu, Dan (Dan)" <dromasca@avaya.com>, Jari Arkko <jari.arkko@piuha.net>
References: <9904FB1B0159DA42B0B887B7FA8119CA5C823BB2@AZ-FFEXMB04.global.avaya.com> <7CC08F3C-4747-4B65-9743-A19CDAE9F940@piuha.net> <9904FB1B0159DA42B0B887B7FA8119CA5C836AB4@AZ-FFEXMB04.global.avaya.com>
In-Reply-To: <9904FB1B0159DA42B0B887B7FA8119CA5C836AB4@AZ-FFEXMB04.global.avaya.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/xmpp/w4KmzWJ8NxTmd9jmtSQaSgDuRUc
Cc: "draft-ietf-xmpp-websocket.all@tools.ietf.org" <draft-ietf-xmpp-websocket.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>, "xmpp@ietf.org" <xmpp@ietf.org>
Subject: Re: [xmpp] [Gen-art] Gen-ART review for draft-ietf-xmpp-websocket-07
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: Tue, 08 Jul 2014 15:22:54 -0000
Hi Dan, As the document shepherd for this spec and the author of RFC 6120, I have one comment below. On 7/8/14, 3:06 AM, Romascanu, Dan (Dan) wrote: > Hi Jari, > > The authors actually responded - see http://www.ietf.org/mail-archive/web/gen-art/current/msg10306.html. > > They pushed back on my #1 - I am still not convinced by their argument (as the protocol does change by adding a different mapping) but I would not block the document for this purpose. > The proposed changes for the rest are fine. Even at the time of RFC 3920 (precursor to RFC 6120), we had one other binding, namely the HTTP binding long-polling binding ("BOSH") defined at http://xmpp.org/extensions/xep-0124.html - this is why in RFC 3920 we explicitly stated that the spec defined only the TCP binding: ### 4.2. Binding to TCP Although there is no necessary coupling of an XML stream to a [TCP] connection (e.g., two entities could connect to each other via another mechanism such as polling over [HTTP]), this specification defines a binding of XMPP to TCP only. In the context of client-to-server communications, a server MUST allow a client to share a single TCP connection for XML stanzas sent from client to server and from server to client. In the context of server-to-server communications, a server MUST use one TCP connection for XML stanzas sent from the server to the peer and another TCP connection (initiated by the peer) for stanzas from the peer to the server, for a total of two TCP connections. ### We repeated this qualification in RFC 6120: ### 3. TCP Binding 3.1. Scope As XMPP is defined in this specification, an initiating entity (client or server) MUST open a Transmission Control Protocol [TCP] connection to the receiving entity (server) before it negotiates XML streams with the receiving entity. The parties then maintain that TCP connection for as long as the XML streams are in use. The rules specified in the following sections apply to the TCP binding. Informational Note: There is no necessary coupling of XML streams to TCP, and other transports are possible. For example, two entities could connect to each other by means of [HTTP] as specified in [XEP-0124] and [XEP-0206]. However, this specification defines only a binding of XMPP to TCP. ### The WebSocket binding specified in this I-D is intended to eventually replace BOSH for reasons explained in RFC 6202. In my opinion, the point of layering via bindings is to not modify or update one binding by defining another binding. Thus I continue to be of the opinion that the WebSocket binding does not update RFC 6120. Peter > > Regards, > > Dan > > >> -----Original Message----- >> From: Jari Arkko [mailto:jari.arkko@piuha.net] >> Sent: Tuesday, July 08, 2014 7:58 AM >> To: Romascanu, Dan (Dan) >> Cc: gen-art@ietf.org; draft-ietf-xmpp-websocket.all@tools.ietf.org; >> xmpp@ietf.org >> Subject: Re: [Gen-art] Gen-ART review for draft-ietf-xmpp-websocket-07 >> >> Thanks for the review, Dan. >> >> I would like to see some thoughts from the editors regarding the two points >> that you raised. >> >> Jari >> >> On 02 Jul 2014, at 09:00, Romascanu, Dan (Dan) <dromasca@avaya.com> >> wrote: >> >>> I am the assigned Gen-ART reviewer for this draft. For background on Gen- >> ART, please see the FAQ at >>> >>> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. >>> >>> Please resolve these comments along with any other Last Call comments >> you may receive. >>> >>> Document: https://datatracker.ietf.org/doc/draft-ietf-xmpp-websocket/ >>> Reviewer: Dan Romascanu >>> Review Date: 7/2/2014 >>> IETF LC End Date: 7/4/2014 >>> IESG Telechat date: 7/10/2014 >>> >>> Summary: ready with minor issues >>> >>> Major issues: >>> >>> None >>> >>> Minor issues: >>> >>> 1. In order to accommodate the Websocket binding this document >> describes several deviations from RFC6120. For example, in Section 3.3 it >> says: >>> The WebSocket XMPP sub-protocol deviates from the standard method of >>> constructing and using XML streams as defined in [RFC6120] by >>> adopting the message framing provided by WebSocket to delineate the >>> stream open and close headers, stanzas, and other top-level stream >>> elements. >>> I am wondering whether it would not be appropriate to reflect this >> in the document header by adding Updates RFC6120 >>> >>> 2. In Section 3.6.1: >>> >>> If the server wishes at any point to instruct the client to move to a >>> different WebSocket endpoint (e.g. for load balancing purposes), the >>> server MAY send a <close/> element and set the "see-other-uri" >>> attribute to the URI of the new connection endpoint (which MAY be for >>> a different transport method, such as BOSH (see [XEP-0124] and >>> [XEP-0206]). >>> >>> I do not understand the usage of MAY in this paragraph. Is there >> another method to move to a different Web socket endpoint that is >> described here or some other place? In not, why is not the first MAY at least >> a SHOULD? The second usage seems to describe a state of facts, so it needs >> not be capitalized at all. >>> >>> >>> Nits/editorial comments: >>> >>> In Section 3.1 I believe that the example should be preceded by some text >> that indicates that this is an example, such as: 'An example of a successful >> handshake and start of session follows:' >>> >>> >>> _______________________________________________ >>> Gen-art mailing list >>> Gen-art@ietf.org >>> https://www.ietf.org/mailman/listinfo/gen-art > > _______________________________________________ > xmpp mailing list > xmpp@ietf.org > https://www.ietf.org/mailman/listinfo/xmpp >
- [xmpp] Gen-ART review for draft-ietf-xmpp-websock… Romascanu, Dan (Dan)
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Jari Arkko
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Lance Stout
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Peter Saint-Andre
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Romascanu, Dan (Dan)
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Peter Saint-Andre
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Dave Cridland
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Dave Cridland
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Kurt Zeilenga
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Richard Barnes
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Romascanu, Dan (Dan)
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Romascanu, Dan (Dan)
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Peter Saint-Andre
- Re: [xmpp] [Gen-art] Gen-ART review for draft-iet… Romascanu, Dan (Dan)