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

Clarify path validation and connection migration #4102

Merged
merged 31 commits into from Sep 22, 2020
Merged
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
c7633d4
Clarify path validation and connection migration
ianswett Sep 16, 2020
7dc91a1
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
d0d6a07
Remove some redundant text
ianswett Sep 16, 2020
74b4347
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
2dc0302
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
e4b8d3e
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
e41f0f9
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
d60deed
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
b5819c8
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
da6ab2c
Remove a duplicative paragraph
ianswett Sep 16, 2020
519a0bb
Move two sentences up
ianswett Sep 16, 2020
d9fbe09
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
b8a37ed
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
850e793
Update draft-ietf-quic-transport.md
ianswett Sep 16, 2020
2aa215b
Clarify what limits the number of paths to probe
ianswett Sep 16, 2020
49efc7a
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
534da74
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
710ccc1
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
b5b490c
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
8eadab3
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
92c1332
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
6c10492
Update draft-ietf-quic-transport.md
ianswett Sep 17, 2020
28bc0a2
Update draft-ietf-quic-transport.md
ianswett Sep 18, 2020
4820145
Update draft-ietf-quic-transport.md
ianswett Sep 22, 2020
f33028f
Update draft-ietf-quic-transport.md
ianswett Sep 22, 2020
01d5680
Restore the paragraph and make a tweak
ianswett Sep 22, 2020
026e0e3
Found the mostly duplicative paragraph in "Path Validation"
ianswett Sep 22, 2020
acb5e7a
Update draft-ietf-quic-transport.md
ianswett Sep 22, 2020
8e05bf5
Update draft-ietf-quic-transport.md
ianswett Sep 22, 2020
2f0bf6f
Apply suggestions from Jana's code review
martinthomson Sep 22, 2020
f617749
Wrap
martinthomson Sep 22, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
21 changes: 8 additions & 13 deletions draft-ietf-quic-transport.md
Expand Up @@ -2160,15 +2160,16 @@ frames used for path validation. In particular, an endpoint can include
PADDING with a PATH_CHALLENGE for Path Maximum Transfer Unit (PMTU) discovery
martinthomson marked this conversation as resolved.
Show resolved Hide resolved
(see {{pmtud}}); it can also include a PATH_CHALLENGE with its own PATH_RESPONSE.

When probing a new path, an endpoint might want to ensure that its peer has an
unused connection ID available for responses. The endpoint can send
NEW_CONNECTION_ID and PATH_CHALLENGE frames in the same packet. This ensures
that an unused connection ID will be available to the peer when sending a
response.
An endpoint uses a new connection ID for probes sent from a new local address
(See {{migration-linkability}}). So when probing a new path, an endpoint
martinthomson marked this conversation as resolved.
Show resolved Hide resolved
expecting responses on the new local address needs to ensure that its peer has
an unused connection ID. The endpoint can send NEW_CONNECTION_ID and
PATH_CHALLENGE frames in the same packet. This ensures that an unused
connection ID will be available to the peer when sending a response.

An endpoint can choose to simultaneously probe multiple paths. The number
of simultaneous paths used for probes is limited by the number of extra
Connection IDs its peer has previously supplied, since each new local address
of simultaneous paths used for probes is limited by the number of extra
Connection IDs its peer has previously supplied, since each new local address
martinthomson marked this conversation as resolved.
Show resolved Hide resolved
used for a probe requires a previously unused Connection ID.
martinthomson marked this conversation as resolved.
Show resolved Hide resolved


Expand Down Expand Up @@ -2298,12 +2299,6 @@ local address. Failure of path validation simply means that the new path is not
usable for this connection. Failure to validate a path does not cause the
connection to end unless there are no valid alternative paths available.

An endpoint uses a new connection ID for probes sent from a new local address;
see {{migration-linkability}} for further discussion. An endpoint that expects
to receive traffic on the new local address needs to ensure that at least one
new connection ID is available at the peer. That can be achieved by including
a NEW_CONNECTION_ID frame in the probe.

PATH_CHALLENGE, PATH_RESPONSE, NEW_CONNECTION_ID, and PADDING frames are
"probing frames", and all other frames are "non-probing frames". A packet
containing only probing frames is a "probing packet", and a packet containing
Expand Down