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

Retransmission Instructions do not include NEW_TOKEN #2871

Closed
martinduke opened this issue Jul 3, 2019 · 2 comments · Fixed by #2961
Closed

Retransmission Instructions do not include NEW_TOKEN #2871

martinduke opened this issue Jul 3, 2019 · 2 comments · Fixed by #2961
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@martinduke
Copy link
Contributor

Sec 13.2 of QUIC Transport explains what to do for the loss of a packet for every stream type except NEW_TOKEN.

IMO it's nice but not critical to repair this loss, but I care less about how we resolve it than that we write it up.
@martinthomson martinthomson added -transport editorial An issue that does not affect the design of the protocol; does not require consensus. labels Jul 8, 2019
martinthomson added a commit that referenced this issue Aug 9, 2019
@martinthomson
Repair rules for NEW_TOKEN
85bcedf 
Closes #2871.
@martinthomson martinthomson mentioned this issue Aug 9, 2019
Merged
@ianswett
Copy link
Contributor

Somewhat related question: Does this section fit better in the recovery draft?

@martinthomson
Copy link
Member

I don't think so. This talks about the repair semantics of these frames, not the process of detecting loss. Recovery doesn't care about the distinction between PING and STREAM when it comes to repair.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Repair rules for NEW_TOKEN quicwg/base-drafts
3 participants
@martinthomson @ianswett @martinduke