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.

Sign up for GitHub

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

Jump to bottom

Simplify stateless retry #817

Merged
merged 4 commits into from Oct 10, 2017
Merged

Simplify stateless retry #817

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7edf4e0
Simplify stateless retry. There's no reason for it to contain ACKs
ekr Oct 2, 2017
03db4e3
Revise text a bit
ekr Oct 2, 2017
fcb91e3
Grrr... Line length
ekr Oct 2, 2017
372c1e6
capitalize STREAM
martinthomson Oct 10, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion draft-ietf-quic-transport.md
View file
Open in desktop
Expand Up @@ -621,7 +621,7 @@ the flow, but other transport state MUST be discarded.
The payload of the Server Stateless Retry packet contains a single
STREAM frame on stream 0 with offset 0 containing the server's
cryptographic stateless retry material. It MUST NOT contain any other
frames. Any future stream frames from the server will also start at
frames. Any future STREAM frames from the server will also start at
stream offset 0.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: STREAM frames. Also, this somewhat implies that ALL STREAM frames will have an offset of 0. I think that you mean that the NEXT STREAM frame starts at an offset of 0.



Expand Down