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

recovery-31 erronous references to external docs #4176

Closed
gloinul opened this issue Oct 2, 2020 · 1 comment · Fixed by #4180
Closed

recovery-31 erronous references to external docs #4176

gloinul opened this issue Oct 2, 2020 · 1 comment · Fixed by #4180
Labels
-recovery ietf-lc An issue that was raised during IETF Last Call.

Comments

@gloinul
Copy link
Contributor

gloinul commented Oct 2, 2020

So there are a couple of section references in the doc that points the wrong section in other documents.

  • Section 6.2.1: "A sender SHOULD restart its PTO timer every time an ack-eliciting packet is sent or acknowledged, when the handshake is confirmed (Section 4.1.2 of [QUIC-TLS]), or when Initial or Handshake keys are discarded (Section 9 of [QUIC-TLS])", should that last ref be to section 4.9 in QUIC-TLS?"

  • Section 6.4: "see Section 4.8 of [QUIC-TLS])" This is now Section 4.9.

  • Section B.2: max_datagram_size: The sender's current maximum payload size. Does not include UDP or IP overhead. The max datagram size is used for congestion window computations. An endpoint sets the value of this variable based on its PMTU (see Section 14.1 of [QUIC-TRANSPORT]), with a minimum value of 1200 bytes." Should that Section 14.1 reference be to 14.2.

@larseggert larseggert added -recovery ietf-lc An issue that was raised during IETF Last Call. labels Oct 2, 2020
@larseggert larseggert added this to Triage in Late Stage Processing via automation Oct 2, 2020
@janaiyengar
Copy link
Contributor

Thank you for catching these, @gloinul.

Late Stage Processing automation moved this from Triage to Issue Handled Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-recovery ietf-lc An issue that was raised during IETF Last Call.
Projects
Late Stage Processing
  
Issue Handled
Development

Successfully merging a pull request may close this issue.

3 participants