-
Notifications
You must be signed in to change notification settings - Fork 203
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
Add stream offset limit of 2^64. #350
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small suggestions.
draft-ietf-quic-transport.md
Outdated
@@ -1182,7 +1182,8 @@ The STREAM frame contains the following fields: | |||
|
|||
* Offset: A variable-sized unsigned number specifying the byte offset in the | |||
stream for the data in this STREAM frame. The first byte in the stream has an | |||
offset of 0. | |||
offset of 0. The largest offset delivered on a stream---the sum of the | |||
re-constructed offset and data length---MUST be lesser than 2^64. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lesser than -> less than, here and below?
draft-ietf-quic-transport.md
Outdated
as an ordered byte-stream. Data received out of order MUST be buffered for | ||
later delivery, as long as it is not in violation of the receiver's flow control | ||
The largest offset delivered on a stream MUST be lesser than 2^64. A receiver | ||
MUST ensure that received stream data is delivered to the application as an |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: How about "MUST ensure that received stream data is available to the application as an ordered byte-stream."
draft-ietf-quic-transport.md
Outdated
@@ -1182,7 +1182,8 @@ The STREAM frame contains the following fields: | |||
|
|||
* Offset: A variable-sized unsigned number specifying the byte offset in the | |||
stream for the data in this STREAM frame. The first byte in the stream has an | |||
offset of 0. | |||
offset of 0. The largest offset delivered on a stream---the sum of the | |||
re-constructed offset and data length---MUST be lesser than 2^64. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: don't use ---
No description provided.