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

Commentary on H3 frames and data_moved events #280

Open
LPardue opened this issue Jan 13, 2023 · 0 comments · May be fixed by #363
Open

Commentary on H3 frames and data_moved events #280

LPardue opened this issue Jan 13, 2023 · 0 comments · May be fixed by #363

Comments

@LPardue
Copy link
Member

LPardue commented Jan 13, 2023

The DATA frame section includes some commentary on H3, frames and streams. In reviewing a PR, I made the following observations:

This seems like it can be genericised and moved somewhere else: there is no tight coupling between H3 and QUIC frame boundaries meaning a H3 frames could span many packets, or many frames could be in a single packet.

Implementations can deploy streaming strategies to any H3 frame parsing. I agree that DATA frames are the most likely, but really any frame could be larger than a packet.

Furthermore, due to varint H3 frame fields and the lack of preserved boundaries, some implementations might generate data_moved events before frame_parsed events. Cloudflare quiche does this, for example.

Originally posted by @LPardue in #274 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant