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

PATH_CHALLENGE timer period #2910

Closed
gorryfair opened this issue Jul 18, 2019 · 2 comments · Fixed by #3339
Closed

PATH_CHALLENGE timer period #2910

gorryfair opened this issue Jul 18, 2019 · 2 comments · Fixed by #3339
Assignees
Labels
-recovery -transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@gorryfair
Copy link
Contributor

Section 9.4 of draft-ietf-quic-transport:
An endpoint might set a
separate timer when a PATH_CHALLENGE is sent, which is cancelled when
the corresponding PATH_RESPONSE is received. If the timer fires
before the PATH_RESPONSE is received, the endpoint might send a new
PATH_CHALLENGE, and restart the timer for a longer period of time.

GF: Possibly so, but what value has this timeout?

  • To avoid potential congestion collapse, I think this needs to be conservative enough to deal with paths with significant RTT variance, or where there is a change of base RTT. 1 sec, then 3 seconds, and then exponentially backed-off would be expected for TCP SYNs, and I suggest could be appropriate.
@ianswett
Copy link
Contributor

The suggestion is to move some of this text to recovery(ie: Handshakes and New Paths) and reference that text from transport.

@ianswett
Copy link
Contributor

As Jana pointed out, this specifies a separate timer, which fits fine with the model that there's one timer per path.

@ianswett ianswett added -recovery editorial An issue that does not affect the design of the protocol; does not require consensus. labels Jul 21, 2019
@mnot mnot added this to Editorial Issues in Late Stage Processing Nov 25, 2019
ianswett added a commit that referenced this issue Jan 13, 2020
Late Stage Processing automation moved this from Editorial Issues to Text Incorporated Feb 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-recovery -transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
Late Stage Processing
  
Issue Handled
Development

Successfully merging a pull request may close this issue.

2 participants