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

Support IPv6 tracing for ICMP and UDP #35

Closed
fujiapple852 opened this issue Apr 25, 2022 · 1 comment · Fixed by #184
Closed

Support IPv6 tracing for ICMP and UDP #35

fujiapple852 opened this issue Apr 25, 2022 · 1 comment · Fixed by #184
Labels
enhancement New feature or request icmp udp
Milestone
0.5.0

Comments

@fujiapple852
Copy link
Owner

fujiapple852 commented Apr 25, 2022 •
edited
Loading

Support for tracing with the IPv6 protocol.
@fujiapple852 fujiapple852 added the enhancement New feature or request label Apr 25, 2022
@fujiapple852 fujiapple852 mentioned this issue May 9, 2022
Closed
@fujiapple852 fujiapple852 added this to the 0.5.0 milestone May 17, 2022
@fujiapple852 fujiapple852 self-assigned this May 18, 2022
@fujiapple852 fujiapple852 mentioned this issue May 23, 2022
Merged
@fujiapple852
Copy link
Owner Author

fujiapple852 commented May 24, 2022 •
edited
Loading

The current implementation for udp over IPv6 uses socket(AF_INET6, SOCK_DGRAM, IPPROTO_UDP) which produces incorrect UDP checksums on (at least) Linux.

MTR uses socket(AF_INET6, SOCK_RAW, IPPROTO_UDP) for this case (the claimed fallback to socket(AF_INET6, SOCK_DGRAM, IPPROTO_UDP) does not appear to work).

To support UDP over IPv6 will likely require a move back to using a raw socket for UDP tracing.

See #192 for follow ups.

@fujiapple852 fujiapple852 removed their assignment May 24, 2022
@fujiapple852 fujiapple852 changed the title IPv6 Support IPv6 tracing for ICMP May 24, 2022
@fujiapple852 fujiapple852 changed the title Support IPv6 tracing for ICMP Support IPv6 tracing for ICMP and UDP May 24, 2022
@fujiapple852 fujiapple852 mentioned this issue May 24, 2022
Closed
@fujiapple852 fujiapple852 mentioned this issue Nov 2, 2023
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request icmp udp
Projects
None yet
Milestone
0.5.0
Development

Successfully merging a pull request may close this issue.

Feat ipv6 fujiapple852/trippy
1 participant
@fujiapple852