ietf-wg-webtrans / draft-ietf-webtrans-http3

Internet Drafts for WebTransport
Other
40 stars 13 forks source link

Clarify long term plans for clients to signal support for WebTransport #130

Closed ekinnear closed 8 months ago

ekinnear commented 1 year ago

Long term, once this has shipped as RFC and all browsers have dropped support for pre-RFC drafts, is the plan to have clients send SETTINGS_WEBTRANSPORT_MAX_SESSIONS forever or to stop sending it? If the goal is to only do this during the draft period, we should have "to be removed by RFC editor" notes explaining that

Originally posted by @DavidSchinazi in https://github.com/ietf-wg-webtrans/draft-ietf-webtrans-http3/issues/129#issuecomment-1585130081

ekinnear commented 1 year ago

Why not use upgrade tokens and update extended connect to support negotiation? Changing semantics of HTTP so that the server can initiate bidirectional streams, that needs to be in a hop-by-hop manner, so a SETTING is needed

Proposal: Server sends SETTINGS_WEBTRANSPORT_MAX_SESSIONS, client doesn't send any settings, but it uses extended connect to ask to use the WebTransport that the server said it supported. If we change the version later, we use a new upgrade token and use a different setting to indicate support, letting the client still pick.

LPardue commented 1 year ago

I like this proposal

ekinnear commented 1 year ago
DavidSchinazi commented 1 year ago

Chair: discussed in editor's meeting. Let's discuss at 117. @ekinnear to write slide

DavidSchinazi commented 11 months ago

Chair: discussed at 117. This was quite confused. We see two proposals:

Please keep discussing on this issue

ekinnear commented 10 months ago

Discussed in editor's meeting:

SETTINGS

Server advertises, client chooses

HTTP syntax is extended by client-initiated bidirectional streams, which is why this matters more than it would otherwise. This raises the question if we should have included a "fake" very high length for the "frame that runs to the end of the stream" so that a parser that's doing the wrong thing won't see the session ID and try to parse out that number of bytes.

Conclusion: Leave this as-is.

DavidSchinazi commented 9 months ago

Chair: discussed in editor's meeting. Does anyone object to leaving this as-is and closing this issue?

DavidSchinazi commented 8 months ago

Chair: closing due to no objections