Skip to content

New path examples #574

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

New path examples #574

wants to merge 1 commit into from

Conversation

mirjak
Copy link
Collaborator

@mirjak mirjak commented Jun 13, 2025

fixes #547

@mirjak mirjak mentioned this pull request Jun 13, 2025
Open
@mirjak mirjak requested review from huitema and martinthomson June 13, 2025 13:32
huitema
huitema approved these changes Jun 15, 2025
View reviewed changes
Copy link
Contributor

@huitema huitema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. I think that removes the issues with the previous version of this diagram.

Yanmei-Liu
Yanmei-Liu approved these changes Jun 16, 2025
View reviewed changes
Copy link
Contributor

@Yanmei-Liu Yanmei-Liu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

martinthomson
martinthomson approved these changes Jun 16, 2025
View reviewed changes
Copy link
Member

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems clearer.

draft-ietf-quic-multipath.md
Comment on lines +392 to +393
<-- 1-RTT[Y]: DCID=C0, PATH_NEW_CONNECTION_ID[S1, Seq=0,
PathID=1], PATH_ACK[PathID=0, PN=X]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<-- 1-RTT[Y]: DCID=C0, PATH_NEW_CONNECTION_ID[S1, Seq=0,
PathID=1], PATH_ACK[PathID=0, PN=X]
<-- 1-RTT[Y]: DCID=C0,
PATH_NEW_CONNECTION_ID[S1, Seq=0, PathID=1],
PATH_ACK[PathID=0, PN=X]

Otherwise, the wrapping is pretty hard to follow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinthomson martinthomson martinthomson approved these changes

@Yanmei-Liu Yanmei-Liu Yanmei-Liu approved these changes

@huitema huitema huitema approved these changes

Assignees
No one assigned
Labels
editorial
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Is the default path path ID 0?
4 participants
@mirjak @martinthomson @Yanmei-Liu @huitema