RE: New Version Notification for draft-mcmanus-httpbis-h2-websockets-00.txt

Hi Patrick,

Interesting, thanks.

Some comments:

1) In section 3:


A sender MUST NOT send a ENABLE_CONNECT_PROTOCOL parameter with the
value of 0 after previously sending a value of 1.


What should the client do in such a situation? It seems overzealous to call this a protocol error, maybe we could

just say that since the initial/default is 0, a change to 1 is sticky and therefore the client MUST ignore any

ENABLE_CONNECT_PROTOCOL with a value of 0. A different way to look at this is that as the document stands,

sending ENABLE_CONNECT_PROTOCOL with a value of 0 is meaningless.

1) In section 7:


The use of a pseudo-header is something that is connection
specific and HTTP/2 does not ever allow to be created outside of
the protocol stack.


I think I understand the intent of this paragraph but I'm not sure its qualified by anything. Is there something that RFC 7540 says on the matter, or is this based on your observation of H2 implementation stacks?

2) In section 8 the error code 0x8 is reserved, however the HTTP/2 Settings registry indicates 0x7 is free. Is this skip over intended?

Cheers
Lucas


________________________________
From: Patrick McManus [mcmanus@ducksong.com]
Sent: 15 October 2017 15:12
To: HTTP Working Group; hybi
Subject: Fwd: New Version Notification for draft-mcmanus-httpbis-h2-websockets-00.txt

FYI - also see https://github.com/mcmanus/draft-h2ws/blob/master/README.md

Comments, expressions of interest, etc are very welcome.


---------- Forwarded message ----------
From: <internet-drafts@ietf.org<mailto:internet-drafts@ietf.org>>
Date: Sun, Oct 15, 2017 at 10:08 AM
Subject: New Version Notification for draft-mcmanus-httpbis-h2-websockets-00.txt
To: Patrick McManus <mcmanus@ducksong.com<mailto:mcmanus@ducksong.com>>



A new version of I-D, draft-mcmanus-httpbis-h2-websockets-00.txt
has been successfully submitted by Patrick McManus and posted to the
IETF repository.

Name:           draft-mcmanus-httpbis-h2-websockets
Revision:       00
Title:          Bootstrapping WebSockets with HTTP/2
Document date:  2017-10-15
Group:          Individual Submission
Pages:          7
URL:            https://www.ietf.org/internet-drafts/draft-mcmanus-httpbis-h2-websockets-00.txt
Status:         https://datatracker.ietf.org/doc/draft-mcmanus-httpbis-h2-websockets/
Htmlized:       https://tools.ietf.org/html/draft-mcmanus-httpbis-h2-websockets-00
Htmlized:       https://datatracker.ietf.org/doc/html/draft-mcmanus-httpbis-h2-websockets-00


Abstract:
   This document defines a mechanism for running the WebSocket Protocol
   [RFC6455] over a single stream of an HTTP/2 connection.




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<http://tools.ietf.org>.

The IETF Secretariat