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

After planned migration, connection is linkable to prev. path #1803

Closed
nemethf opened this issue Apr 3, 2024 · 1 comment · Fixed by #1857
Closed

After planned migration, connection is linkable to prev. path #1803

nemethf opened this issue Apr 3, 2024 · 1 comment · Fixed by #1857
Labels
bug Something isn't working

Comments

@nemethf
Copy link
Contributor

nemethf commented Apr 3, 2024

The server switches to a new CID, but it sends a path-challenge on the old path with the new CID. So it is still possible to trace back the connection to the old path after the client switches to a new socket.

See #1800 for breaking linkability on the client side.
nemethf added a commit to nemethf/quinn that referenced this issue Apr 3, 2024
@nemethf
Break linkability on client-side after planned migration
523c7d9 
(See bug quinn-rs#1803 for the server-side case.)
@Ralith Ralith added the bug Something isn't working label Apr 3, 2024
Ralith pushed a commit to nemethf/quinn that referenced this issue Apr 4, 2024
@nemethf @Ralith
Break linkability on client-side after planned migration
23cc3e3 
(See bug quinn-rs#1803 for the server-side case.)
Ralith pushed a commit to nemethf/quinn that referenced this issue Apr 4, 2024
@nemethf @Ralith
Break linkability on client-side after planned migration
43bb95d 
(See bug quinn-rs#1803 for the server-side case.)
Ralith pushed a commit that referenced this issue Apr 4, 2024
@nemethf @Ralith
Break linkability on client-side after planned migration
e28a7f2 
(See bug #1803 for the server-side case.)
gabrik pushed a commit to gabrik/quinn that referenced this issue Apr 4, 2024
@nemethf @gabrik
Break linkability on client-side after planned migration
cac0902 
(See bug quinn-rs#1803 for the server-side case.)
@djc djc mentioned this issue May 6, 2024
Closed
7 tasks
@Ralith Ralith mentioned this issue May 10, 2024
Merged
@Ralith
Copy link
Collaborator

Ralith commented May 10, 2024

#1857 should sort this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix previous path validation allowing flows to be linked across migration quinn-rs/quinn
2 participants
@Ralith @nemethf