Good luck.
Hi, It’s also advisable to start with a small contribution and have that reviewed to get feedback on coding style etc and to get to know the process. A small patch stands a much better chance of getting a fast review than a large one. Best regards Anders Broman Basically you're right. If it's not included, it's because no one pushed a change for inclusion. Wireshark includes as many protocols as possible and zookeeper would be definitively accepted. Before starting, be sure that something doesn't exist out there: you could start for someone else's code (according to its licence, of course) and not start from scratch. When submitting the dissector, please be sure to add sample captures for testing. To add them, just file a bug on bugzilla with your attachments, and add a reference "Bug: XXX" in your commit message that will automatically link the bugzilla page. How do I get a dissector for Zookeeper's protocol included by default in Wireshark? I searched but couldn't find anything about what's required to "promote" a dissector to be packaged with wireshark... I assume it's more than just code, that there's some process for deciding whether a protocol is popular enough that a dissector for it could be included in core. Zookeeper is an open-source database for reliably storing metadata. Many popular open-source distributed systems rely heavily on it, including Kafka, Hadoop, Druid, etc. Given this popularity, I suspect the only reason it's not bundled into Wireshark is no one has had the time to write it, but wanted to confirm before I start hacking on it.
|