Re: [dispatch] I-D Action: draft-pd-dispatch-msrp-websocket-03.txt

Ben Campbell <ben@nostrum.com> Mon, 20 January 2014 04:32 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: dispatch@ietfa.amsl.com
Delivered-To: dispatch@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 09A6D1A0043 for <dispatch@ietfa.amsl.com>; Sun, 19 Jan 2014 20:32:39 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.036
X-Spam-Level:
X-Spam-Status: No, score=-1.036 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_MISMATCH_COM=0.553, HOST_MISMATCH_NET=0.311] autolearn=no
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 WzCIOJPb0-lB for <dispatch@ietfa.amsl.com>; Sun, 19 Jan 2014 20:32:36 -0800 (PST)
Received: from shaman.nostrum.com (nostrum-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:267::2]) by ietfa.amsl.com (Postfix) with ESMTP id 672BF1A0032 for <dispatch@ietf.org>; Sun, 19 Jan 2014 20:32:36 -0800 (PST)
Received: from [10.0.1.29] (cpe-173-172-146-58.tx.res.rr.com [173.172.146.58]) (authenticated bits=0) by shaman.nostrum.com (8.14.3/8.14.3) with ESMTP id s0K4WMoV014055 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Sun, 19 Jan 2014 22:32:23 -0600 (CST) (envelope-from ben@nostrum.com)
Content-Type: text/plain; charset=windows-1252
Mime-Version: 1.0 (Mac OS X Mail 7.1 \(1827\))
From: Ben Campbell <ben@nostrum.com>
In-Reply-To: <t8ggf2ti82dib0706kka9dx1.1390188532252@email.android.com>
Date: Sun, 19 Jan 2014 22:32:21 -0600
Content-Transfer-Encoding: quoted-printable
Message-Id: <C7680923-6498-428A-A4F8-F057AF383A83@nostrum.com>
References: <20131213005747.777.34301.idtracker@ietfa.amsl.com> <CAHBDyN4tSRO_nYy7_-V4xfmDbF0ZeLJ24_fEOQ1p9Z2BvJyinQ@mail.gmail.com> <7594FB04B1934943A5C02806D1A2204B1D104D91@ESESSMB209.ericsson.se>, <CAEqTk6QcSU+u2nrp3oyoyr6p4diGD2s4-4PhBQW-UP2VdZmsqw@mail.gmail.com> <t8ggf2ti82dib0706kka9dx1.1390188532252@email.android.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>
X-Mailer: Apple Mail (2.1827)
Received-SPF: pass (shaman.nostrum.com: 173.172.146.58 is authenticated by a trusted mechanism)
Cc: DISPATCH <dispatch@ietf.org>, "draft-pd-dispatch-msrp-websocket@tools.ietf.org" <draft-pd-dispatch-msrp-websocket@tools.ietf.org>
Subject: Re: [dispatch] I-D Action: draft-pd-dispatch-msrp-websocket-03.txt
X-BeenThere: dispatch@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DISPATCH Working Group Mail List <dispatch.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dispatch>, <mailto:dispatch-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dispatch/>
List-Post: <mailto:dispatch@ietf.org>
List-Help: <mailto:dispatch-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dispatch>, <mailto:dispatch-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 20 Jan 2014 04:32:39 -0000

On Jan 19, 2014, at 9:28 PM, Christer Holmberg <christer.holmberg@ericsson.com> wrote:

> Hi,
> 
> I see no reason why it should be a separate document, as it should not have any affect on the websocket specific procedures, which is the main scope of the document.

Christer, I assume you mean for a WebSocket MSRP "server" acting on behalf of a WS MSRP client to be able to use CEMA between itself and a third party client? Not between a WebSocket MSRP client and a WebSocket MSRP server, right?

> 
> Regards,
> 
> Christer
> 
> Sent from my Sony Ericsson Xperia arc S
> 
> Peter Dunkley <peter.dunkley@crocodilertc.net> wrote:
> 
> 
> Hello,
> 
> Perhaps the document title should be corrected. MSRP-CEMA is outside of the scope of this document as this document is intended to describe connecting to a WebSocket server that is an MSRP relay.
> 
> I can see no reason why MSRP-CEMA can't be used over WebSockets, but if anyone has an interest in this I think that they should put it in a document of its own.
> 
> 
> Regards,
> 
> Peter
> 
> 
> On 18 January 2014 08:52, Christer Holmberg <christer.holmberg@ericsson.com> wrote:
> Hi,
> 
> 
> 
> I have not followed the work on this draft, so I appologize if the following has been discussed.
> 
> 
> 
> While I do understand that a WS Client has to establish the WebSocket with the Web Server, I don’t understand why we need to mandate the WS Server to be an MSRP Relay. That would e.g. prevent the usage of MSRP-CEMA.
> 
> 
> 
> Regards,
> 
> 
> 
> Christer
> 
> 
> 
> 
> 
> 
> 
> Lähettäjä: dispatch [mailto:dispatch-bounces@ietf.org] Puolesta Mary Barnes
> Lähetetty: 11. tammikuuta 2014 0:59
> Vastaanottaja: DISPATCH
> Kopio: Ben Campbell; draft-pd-dispatch-msrp-websocket@tools.ietf.org
> Aihe: Re: [dispatch] I-D Action: draft-pd-dispatch-msrp-websocket-03.txt
> 
> 
> 
> I have agreed to shepherd this document.  I've reviewed the document in anticipation of doing the PROTO write-up and have the following comments and questions.  Ben Campbell has agreed to do the required expert review and that should be posted within the next week or so.    This is also a good time for anyone in the WG that hasn't previously reviewed this document to review and provide any final comments.  Note, that this document was agreed to be AD sponsored around the IETF-86 timeframe.
> 
> 
> 
> Regards,
> 
> Mary. 
> 
> 
> 
> Review Summary: Almost ready. Comments & questions below.
> 
> 
> 
> 1)  Section 2 & General.  I'm not sure the documented approach for separating normative text from non-normative is quite so helpful.  In general, we expect that in the case of standards track document use RFC 2119 language to indicate normative behaviors.  I think the first sentence is good, but that's not a terminology thing.   I just don't see a lot of value in writing the document this way.  For example, the definitions aren't stated to be non-normative, but I don't see anything normative about the definitions.  I think you could easily title Section 3 as "WebSocket Protocol overview" and that would clearly imply non-normative behavior.  There are also several places in the document in sections that I believe are intended to provide normative behavior, but there is certainly non-normative text in those sections (e.g., section 5.2.2, second paragraph).  I would suggest this document follow the typical (and accepted) style of identifying normative behavior with 2119 language (consistently using upper case for normative behavior and avoiding using 2119 language in cases where alternative words can be substituted).
> 
> 
> 
> 2) Section 5.2.2, 2nd paragraph.  Related to my point above, it's not clear to me this is normative behavior.  I don't think it is since it's referring to existing 4975 behavior. However, I didn't see that the reference given in 4975 relates to the second part of that sentence stating that implementations "should" already be allowing unrecognized transports.  It would be quite useful to have the exact reference here as I was trying to double check this point and I couldn't find it. 
> 
> 
> 
> 3) Section 6.  I'm really puzzled as to why the Connection Keep-alive would be non-normative.  In particular given that 2119 language is clearly being used.  
> 
> 
> 
> 4) Section 7.  Again, I'm puzzled as to why Authentication is considered non-normative. AGain, you have 2119 language in this section.  
> 
> 
> 
> 5) Section 10.1. Since securing the connection is just RECOMMENDED, what are the implications and risks if the MSRP traffic isn't transported over a secure connection? 
> 
> 
> 
> 6) Section 11.  You should change the name of the registry to be the exact name of the IANA registry to avoid any confusion.- i.e.,:
> 
> OLD:
> 
>  registry of WebSocket sub-protocols
> 
> NEW: 
> 
>  WebSocket Subprotocol Name Registry  
> 
> 
> 
> 7) Section 11. There is also a Reference field in that IANA registry. I would suggest you use the same information as the pointer to the Subprotocol Definition (i.e., this RFC). 
> 
> 
> 
> 8) It's typical for documents that are updating existing RFCs to have a section that summarizes the updates to the existing RFCs that are made by this document.  
> 
> 
> 
> 
> 
> 
> 
> On Thu, Dec 12, 2013 at 6:57 PM, <internet-drafts@ietf.org> wrote:
> 
> 
> A New Internet-Draft is available from the on-line Internet-Drafts directories.
> 
> 
>        Title           : The WebSocket Protocol as a Transport for the Message Session Relay Protocol (MSRP)
>        Author(s)       : Peter Dunkley
>                          Gavin Llewellyn
>                          Victor Pascual
>                          Anton Roman
>                          Gonzalo Salgueiro
>        Filename        : draft-pd-dispatch-msrp-websocket-03.txt
>        Pages           : 21
>        Date            : 2013-12-12
> 
> Abstract:
>   The WebSocket protocol enables two-way real-time communication
>   between clients and servers.  This document specifies a new WebSocket
>   sub-protocol as a reliable transport mechanism between MSRP (Message
>   Session Relay Protocol) clients and relays to enable usage of MSRP in
>   new scenarios.  This document normatively updates RFC 4975 and RFC
>   4976.
> 
> 
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-pd-dispatch-msrp-websocket
> 
> There's also a htmlized version available at:
> http://tools.ietf.org/html/draft-pd-dispatch-msrp-websocket-03
> 
> A diff from the previous version is available at:
> http://www.ietf.org/rfcdiff?url2=draft-pd-dispatch-msrp-websocket-03
> 
> 
> 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/
> 
> _______________________________________________
> I-D-Announce mailing list
> I-D-Announce@ietf.org
> https://www.ietf.org/mailman/listinfo/i-d-announce
> Internet-Draft directories: http://www.ietf.org/shadow.html
> or ftp://ftp.ietf.org/ietf/1shadow-sites.txt
> 
> 
> 
> 
> 
> 
> -- 
> Peter Dunkley
> Technical Director
> Crocodile RCS Ltd