Wireshark · Wireshark-dev: Re: [Wireshark-dev] What's the process to get a Zookeeper dissector packaged with wireshark?

[Jeff Widman <jeff@xxxxxxxxxxxxxx>]

Thanks.

Someone already released a Zookeeper lua-based dissector under MIT license:

https://github.com/ixiacom/ATI/blob/master/Apps/Wireshark%20Dissectors/zab.lua

Can it be included in core as a lua-based dissector or should I port it to C first? 

I'm also willing to do any additional work required to move this across the finish line, such as generating bugzilla sample captures, etc.

And thanks everyone for the other tips/doc links on submitting patches to this project. I appreciate the warm welcome.

Cheers,

Jeff

On Wed, Jan 24, 2018 at 12:43 AM, Dario Lombardo <dario.lombardo.ml@xxxxxxxxx> wrote:

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.

Dario.

On Wed, Jan 24, 2018 at 9:12 AM, Jeff Widman <jeff@xxxxxxxxxxxxxx> wrote:

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.

--

Jeff Widman

jeffwidman.com | 740-WIDMAN-J (943-6265) 

<><

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@wireshark.org?subject=unsubscribe

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@wireshark.org?subject=unsubscribe

--

Jeff Widman

jeffwidman.com | 740-WIDMAN-J (943-6265) 

<><