[dispatch] AD Evaluation of draft-pd-dispatch-msrp-websocket

"Ben Campbell" <ben@nostrum.com> Fri, 10 June 2016 22:41 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: dispatch@ietfa.amsl.com
Delivered-To: dispatch@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6172C12B063 for <dispatch@ietfa.amsl.com>; Fri, 10 Jun 2016 15:41:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.325
X-Spam-Level:
X-Spam-Status: No, score=-3.325 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RP_MATCHES_RCVD=-1.426] autolearn=ham autolearn_force=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 rU5rq-2H3YtJ for <dispatch@ietfa.amsl.com>; Fri, 10 Jun 2016 15:41:31 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id EABC012B00B for <dispatch@ietf.org>; Fri, 10 Jun 2016 15:41:30 -0700 (PDT)
Received: from [10.0.1.4] (cpe-66-25-7-22.tx.res.rr.com [66.25.7.22]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id u5AMfTct034463 (version=TLSv1 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Fri, 10 Jun 2016 17:41:30 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-66-25-7-22.tx.res.rr.com [66.25.7.22] claimed to be [10.0.1.4]
From: Ben Campbell <ben@nostrum.com>
To: draft-pd-dispatch-msrp-websocket.all@tools.ietf.org
Date: Fri, 10 Jun 2016 17:41:29 -0500
Message-ID: <9957D411-C669-4490-9198-18F27A5650A7@nostrum.com>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="=_MailMate_E810A3B9-B1DB-4D86-969F-2CA6C9DC86BB_="
X-Mailer: MailMate (1.9.4r5234)
Archived-At: <https://mailarchive.ietf.org/arch/msg/dispatch/jqzP_0XUCT6jG0U3wyZk1q9Afjc>
Cc: DISPATCH <dispatch@ietf.org>
Subject: [dispatch] AD Evaluation of draft-pd-dispatch-msrp-websocket
X-BeenThere: dispatch@ietf.org
X-Mailman-Version: 2.1.17
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: <https://mailarchive.ietf.org/arch/browse/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: Fri, 10 Jun 2016 22:41:32 -0000

Hi,

This is my AD Evaluation of draft-pd-dispatch-msrp-websocket. I have a 
few editorial comments, but nothing that should block going to IETF last 
call. I will request the last call shortly.

Thanks!

Ben.

-----------------------

-1, last paragraph: Consider citing RFC4976 for "MSRP relay". (I also 
suggest citations to 4975 and 4976 when you talk about "normal" clients 
and relays.

-5.1, first paragraph: "...messages MUST be routed
    via an MSRP WebSocket Server"

This seems more a statement of fact than a normative requirement. That 
is, it is an implication of the fact that websocket clients cannot 
directly connect to other websocket clients. Please consider removing 
the upper-case MUST.

- 5.3.1. 4th paragraph (1st after numbered list): s/event/even