w-o-o / httppubsub Goto Github PK

We have

If an RStream is reset,
endpoints MUST reset the XStreams associated with that RStream. If the
RStream is closed, endpoints SHOULD allow the existing XStreams to complete
normally. The RStream SHOULD remain open while communication is ongoing.
Endpoints SHOULD refresh any timeout on the RStream while its associated
XStreams are open.

Along with some later text putting in the requirements for half-closed streams. We may want some descriptive text prior to those requirements that motivates them and explains the conceptual model behind what we're asking people to do.

Current text:

The extension SHOULD be disabled by default. As suggested in {{!RFC7540}},
section 5.5, HTTP/2 compliant implemenations which do not yet support this
extension MUST ignore the unknown ENABLE_XHEADERS setting and XHEADERS
frame.

We should discuss RStreams here as well, since they don't really have a SETTING and don't really need one, but that should be explained.

3. Clients are able to group streams together for routing purposes,
   such that each individual stream group can be used for a different service,
   within the same HTTP/2 connection.

It seems odd to me that we're mentioning groups for the first time as the last item in the list at the end of the introduction, when it seems to have a pretty big effect on what's going on. We should consider setting the stage earlier for what problem we solve with groups.

We currently have the text:

XStream but discovers the destination endpoint does not
support the extension, it MUST reset the stream with
XHEADER_NOT_ENABLED_ERROR.

Should this be a different error from the previously mentioned error for getting an XStream before the settings value?

From #2, there was additional value for intermediaries to be able to properly route everything, we should replace the telemetry sentence with something that focuses on them being able to help with the interactions the client/server are trying to have.