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

The extension is a little weakly defined #229

Closed
LPardue opened this issue Oct 30, 2023 · 0 comments · Fixed by #266
Closed

The extension is a little weakly defined #229

LPardue opened this issue Oct 30, 2023 · 0 comments · Fixed by #266

Comments

@LPardue
Copy link
Member

LPardue commented Oct 30, 2023

This is an editorial matter and subjective. The actual extension gubbins are fine, I just find that the references to the concept of the extenstion is pretty weak.

For instance, the introduction is literally 1 sentence and says just "This document describes a QUIC extension for an endpoint to control its peer's delaying of acknowledgements.". It would be nicer to expand this a little bit to let the reader in on what type of extension we're talking about - for example, datagrams mentions there are frame types https://datatracker.ietf.org/doc/html/rfc9221#section-1 but I suspect that this draft has a little more nuance.

mirjak added a commit that referenced this issue Jan 12, 2024
@mirjak mirjak added has-pr and removed needs-pr labels Jan 12, 2024
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.

2 participants