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
require ACKs and allow PADDING in the Server Stateless Retry packet #882
require ACKs and allow PADDING in the Server Stateless Retry packet #882
Conversation
I'm not in favor of this for the reasons discussed in #879, so we should discuss before merging. |
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.
I like this change, but we should resolve the issue to decide whether to merge.
draft-ietf-quic-transport.md
Outdated
on stream 0 with offset 0 containing the server's cryptographic stateless retry | ||
material. It MUST NOT contain any other frames. The next STREAM frame sent by | ||
the server will also start at stream offset 0. | ||
The payload of the Retry packet contains a STREAM frame on |
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.
I don't think the first two lines here should be changed. Maybe this is just out of date?
This PR needs to be re-targeted to cover the new scope of #627, but it's otherwise good. |
See discussion in quicwg#627 for the stronger language for ACK frames. Closes quicwg#1067. Updates quicwg#627.
Ugh, sorry @marten-seemann -- I meant to send a patch to your commit, but ended up committing to your branch instead. Take a look, and respond if you think it's fine. I'm happy to merge if you agree. |
... RTT measurement is a bonus.
draft-ietf-quic-transport.md
Outdated
The payload of the Retry packet contains at least two frames. It MUST include a | ||
STREAM frame on stream 0 with offset 0 containing the server's cryptographic | ||
stateless retry material. It MUST also include an ACK frame to acknowledge the | ||
client's Initial packet. It MAY additionally include one ore more PADDING |
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.
s/ore/or
LGTM. |
Closes #1067. Updates #627.