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

RFE: ICMPv6 parser #15

Open
daviessm opened this issue Jan 7, 2022 · 6 comments · May be fixed by #16
Open

RFE: ICMPv6 parser #15

daviessm opened this issue Jan 7, 2022 · 6 comments · May be fixed by #16

Comments

@daviessm
Copy link

daviessm commented Jan 7, 2022

I'd like to be able to parse ICMPv6 packets.

@bestouff
Copy link
Owner

bestouff commented Jan 7, 2022

Would you go as far as submitting some code ?

@daviessm
Copy link
Author

daviessm commented Jan 7, 2022

Yes, I'll pick it up myself if/when I get time but I'm equally happy for someone else to get there first!

@bestouff
Copy link
Owner

bestouff commented Jan 7, 2022

Great ! Please submit a PR when you're ready, we'll make sure to review it.

@daviessm
Copy link
Author

daviessm commented Mar 2, 2022

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?

@bestouff
Copy link
Owner

bestouff commented Mar 2, 2022

Well, as long as you produce something useful and correct, even if not complete I have no difficulties accepting a PR.
Could you add some tests with real-life ICMPv6 packets ?

@daviessm
Copy link
Author

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.

@daviessm daviessm linked a pull request May 24, 2022 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants