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

State clearly what lengths mean for stream and datagram data moved #395

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LPardue
Copy link
Member

@LPardue LPardue commented Feb 19, 2024

Addresses part of #375

@rmarx
Copy link
Contributor

rmarx commented Feb 19, 2024

As discussed during editorial meeting:
I'm not the biggest fan of current text since it isn't clear for all the possible uses (e.g., transport to application strips QUIC level stuff but keeps H3 frame headers, while application to user also strips those out etc.). Maybe best to word it in terms of "only the payload is moved to the higher layer" or something like that?

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

Successfully merging this pull request may close these issues.

None yet

2 participants