Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for CLOSE_FRAME #531

Open
aboba opened this issue Jul 25, 2023 · 5 comments
Open

Support for CLOSE_FRAME #531

aboba opened this issue Jul 25, 2023 · 5 comments
Assignees

Comments

@aboba
Copy link
Collaborator

aboba commented Jul 25, 2023

We now have: https://datatracker.ietf.org/doc/draft-ietf-quic-reliable-stream-reset/

What changes (if any) should be made to the API to support this?

@bashi
Copy link
Contributor

bashi commented Jul 25, 2023

Probably the IETF draft needs to be revised to specify the behavior about CLOSE_FRAME first, then the W3C spec refers the behavior? cc/ @vasilvv

@vasilvv
Copy link
Contributor

vasilvv commented Jul 25, 2023

Well, there are two questions:
(1) Use of CLOSE_FRAME internally by WebTransport itself. That would have to be added to the IETF draft.
(2) Providing an API for Web applications to use it. This should not be that hard (though the async nature of stream writes does make it somewhat tricky to do exactly), but I'd wait until someone actually asks for that.

@bashi
Copy link
Contributor

bashi commented Jul 25, 2023

+1. IIUC CLOSE_FRAME is useful to associate streams to a WebTransport session reliably. I'm not sure there are concrete use cases for applications.

@aboba
Copy link
Collaborator Author

aboba commented Jul 25, 2023

For something like partial reliability (e.g. setting a timer and sending a reset if it expires), I don't think reliable reset makes sense. If you're giving up on a frame, you don't care if part of it is sent reliably. So it wasn't clear to me why MoQ would need reliable reset.

@wilaw wilaw added the Discuss at next meeting Flags an issue to be discussed at the next WG working label Aug 14, 2023
@jan-ivar
Copy link
Member

jan-ivar commented Aug 15, 2023

Meeting:

  • Wait and see
  • Wait for underlying concept to materialize

@wilaw wilaw added this to the Future version milestone Sep 20, 2023
@wilaw wilaw removed the Discuss at next meeting Flags an issue to be discussed at the next WG working label Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants