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

Confusing Text in Stateless Reset #1025

Closed
mcmanus opened this issue Dec 18, 2017 · 0 comments
Closed

Confusing Text in Stateless Reset #1025

mcmanus opened this issue Dec 18, 2017 · 0 comments
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@mcmanus
Copy link
Contributor

mcmanus commented Dec 18, 2017

After the first short header octet and optional connection ID, the server includes the value of the Stateless Reset Token that it included in its transport parameters.

After the Packet Number, the server pads the message with an arbitrary number of octets containing random values.

Finally, the last 16 octets of the packet are set to the value of the Stateless Reset Token.

The first para seems to say to me the 16 byte token goes before the bad, the last para says it goes after that pad at the end of the packet. The diagram is pretty clear the latter is correct. I suggest removing? the first para.

martinthomson added a commit that referenced this issue Dec 18, 2017
The location of the stateless reset token is made less clear by this text now that it has been moved to the end.

Closes #1025.
@martinthomson martinthomson added editorial An issue that does not affect the design of the protocol; does not require consensus. -transport labels Jan 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

No branches or pull requests

2 participants