-
Notifications
You must be signed in to change notification settings - Fork 9
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
RFE: ICMPv6 parser #15
Comments
Would you go as far as submitting some code ? |
Yes, I'll pick it up myself if/when I get time but I'm equally happy for someone else to get there first! |
Great ! Please submit a PR when you're ready, we'll make sure to review it. |
I've started work on this at https://github.com/daviessm/pktparse-rs/tree/icmpv6 - currently it only parses the "type" and "code" fields (and isn't fully tested). ICMPv6 is a big beast and I doubt I'll ever be able to code the whole lot of possible options. At what point would you like me to submit a PR? |
Well, as long as you produce something useful and correct, even if not complete I have no difficulties accepting a PR. |
I've added tests for echo request and reply. Can you see whether this is the style you'd like it to be written in? Also, the echo payload data isn't parsed (because it can't be) and is returned as-is. It seems that the ICMPv4 parser does the same thing. |
I'd like to be able to parse ICMPv6 packets.
The text was updated successfully, but these errors were encountered: