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

unblock Connection Migration testing #266

Open
1 task
toidiu opened this issue May 24, 2021 · 4 comments
Open
1 task

unblock Connection Migration testing #266

toidiu opened this issue May 24, 2021 · 4 comments

Comments

@toidiu
Copy link

toidiu commented May 24, 2021

Creating a issue to track the effort required to enable Connection Migration interop testing.

Summary:

The next three tests are disabled due to Wireshark not being able to decrypt packets sent on the new path.

A comment in the repo states that tests related to connection migration are disabled because of missing support from Wireshark.

I found this issue in the Wireshark repo but i was last updated 6+ months ago: https://gitlab.com/wireshark/wireshark/-/issues/16915

Criteria for success:

@toidiu
Copy link
Author

toidiu commented May 24, 2021

Open questions I have:

  • Is anyone currently working on this or is there a plan to pick up this feature?
  • How are implementations currently testing that connection migration works (I imagine a custom integration testing framework)?

@marten-seemann
Copy link
Collaborator

Can you try running these disabled tests with a current version of Wireshark? There's still a chance it got fix in the mean time, and the tracking issue wasn't properly updated.
Otherwise, we'd probably have to poke the Wireshark guys to fix their code ;)

@toidiu
Copy link
Author

toidiu commented Jun 3, 2021

Sure, let me enable the test and see if a recent version of Wireshark now supports the feature.

@johnthacker
Copy link

You should be able to enable those tests now (though some implementations will fail - an example posted to our bug did not issue a PATH_CHALLENGE when the port but not the IP address changed.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants