Re: [secdir] Reviews of draft-ietf-xmpp-websocket-07

Lance Stout <lance@andyet.net> Fri, 04 July 2014 02:30 UTC

Return-Path: <lance@andyet.net>
X-Original-To: secdir@ietfa.amsl.com
Delivered-To: secdir@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D92761B297B for <secdir@ietfa.amsl.com>; Thu, 3 Jul 2014 19:30:49 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=unavailable
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 hcAm3buXJDFG for <secdir@ietfa.amsl.com>; Thu, 3 Jul 2014 19:30:48 -0700 (PDT)
Received: from mail-pd0-f180.google.com (mail-pd0-f180.google.com [209.85.192.180]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0643C1B298B for <secdir@ietf.org>; Thu, 3 Jul 2014 19:30:46 -0700 (PDT)
Received: by mail-pd0-f180.google.com with SMTP id fp1so1159697pdb.25 for <secdir@ietf.org>; Thu, 03 Jul 2014 19:30:46 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:content-type:mime-version:subject:from :in-reply-to:date:cc:message-id:references:to; bh=5jp7ssa4ZDuyUnV5HqgJtNJeBetX4mrY5ee+CL3iIVw=; b=VXVqIIvkOD6nlY2XfXcRc0WYwdcgpZD/OStJgPGdCk6WoZbd2mYHqHlwDxRksA4Hbb +Lwk3vtxYKmQY4kJ/Zy9f2xnsWQA5Pd46dtP7CC4PMbNap9EIZU+2nuckbikNKomJZV1 WJjm46nxumspy9JVFr2nrZ7wdM7J/FWv4B2PuSAaIXHSExsTh6a8MzH3WDOrsU7HjQaN K3YyRojhdg42sUNer8VUM0rgqTkHk6oPROBxdIbQAO3hrlO2t2GfExMudaLQHYjw4bkI FJhVruhExf0YMe+FPMseYQ29elLCtozPtMJ78ykAaiik+aOyXGbYEqTZca/QTKYg/bG6 PSJA==
X-Gm-Message-State: ALoCoQmC9Sk5TslCYdnUHoL2zKepR8vbH7YOyPOyeN5uILPwetxkwOtE/M99ClRiWHcHhEpbkmam
X-Received: by 10.70.15.225 with SMTP id a1mr7552927pdd.13.1404441046639; Thu, 03 Jul 2014 19:30:46 -0700 (PDT)
Received: from [10.0.1.172] (68-186-83-170.dhcp.knwc.wa.charter.com. [68.186.83.170]) by mx.google.com with ESMTPSA id h15sm6709498pdl.5.2014.07.03.19.30.44 for <multiple recipients> (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 03 Jul 2014 19:30:45 -0700 (PDT)
Content-Type: multipart/signed; boundary="Apple-Mail=_1F4FC48B-2C42-4B9B-A09F-838E4C7BB53A"; protocol="application/pkcs7-signature"; micalg=sha1
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.2\))
From: Lance Stout <lance@andyet.net>
In-Reply-To: <53b587f5.09d5440a.44eb.2126@mx.google.com>
Date: Thu, 3 Jul 2014 19:30:43 -0700
Message-Id: <46B2A4E0-236E-4B1C-8060-C8641E0CA012@andyet.net>
References: <CADajj4YJVxE8fh1iuZQ0qTPGrvnF_N_ywYBsouifN2jv-UqJZQ@mail.gmail.com> <53b587f5.09d5440a.44eb.2126@mx.google.com>
To: "iesg@ietf.org" <iesg@ietf.org>
X-Mailer: Apple Mail (2.1878.2)
Archived-At: http://mailarchive.ietf.org/arch/msg/secdir/WzVzvcZb7m286xYFg_f9-K81xvs
X-Mailman-Approved-At: Fri, 04 Jul 2014 07:01:54 -0700
Cc: ops-dir@ietf.org, "secdir@ietf.org" <secdir@ietf.org>, ops-ads@tools.ietf.org, "gen-art@ietf.org" <gen-art@ietf.org>, "Romascanu, Dan \(Dan\)" <dromasca@avaya.com>, draft-ietf-xmpp-websocket.all@tools.ietf.org, "draft-ietf-xmpp-websocket@tools.ietf.org" <draft-ietf-xmpp-websocket@tools.ietf.org>
Subject: Re: [secdir] Reviews of draft-ietf-xmpp-websocket-07
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>, <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>, <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 04 Jul 2014 02:30:50 -0000

Closing in on the end of LC for draft-ietf-xmpp-websocket-07, so replying to the
feedback generated so far. We'll publish a -08 draft addressing these issues,
which have all been minor or editorial points.




On Jul 3, 2014, at 9:39 AM, <magnusn@gmail.com>; <magnusn@gmail.com>; wrote:

> Should “when TLS is used, it MUST be enabled the WebSocket layer ” have read
> “when TLS is used, it MUST be enabled at the WebSocket layer ” ?


Yes, that is the intended wording.




On Jul 2, 2014, at 6:00 AM, Romascanu, Dan (Dan) <dromasca@avaya.com>; wrote:

> 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


This is a separate binding from the TCP binding defined in RFC6120, so I don't
think saying Updates RFC6120 would be accurate. Nothing in RFC6120 is modified
by this document.


> 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.

That is the only method, so I agree that can be a SHOULD, and also agree on the
second point.


> 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:’

+1, will add that.





On Jun 25, 2014, at 11:55 PM, Juergen Schoenwaelder
<j.schoenwaelder@jacobs-university.de>; wrote:

> - Sec 1: The term 'raw socket' can be potentially mis-understood, perhaps simply
> remove 'over row sockets' completely (I think the message of the sentence
> remains intact without these words).

+1 will change

> - Sec 3.1: The text says that both client and server MUST have |xmpp| in the
> list of protocols for the |Sec-WebSocket-Protocol| header. The text does not
> detail what happens if this is not the case. Is there be a defined behavior if
> this protocol negotiation fails?

Good catch, RFC6455 doesn't describe what to do in that case, so we'll need to
address it.

If the server does not reply with 'xmpp' in the Sec-WebSocket-Protocol handshake
reply, then the client MUST close the connection.


> - Sec 3.6.1: There is a closing parenthesis missing at the end of the first
> paragraph.

Noted, will fix.




- Lance