kaitoy / pcap4j

A Java library for capturing, crafting, and sending packets.
https://www.pcap4j.org/
Other
1.17k stars 311 forks source link

evaluating pcap4j #9

Closed dzonekl closed 10 years ago

dzonekl commented 11 years ago

Hi Kaitoy,

Just to let you know, I am evaluating pcap4j (and other pcap libs) for integration in netXStudio. (www.netxstudio.com).

I will need to process various telecommunications protocols (sigtran, MTP3, MAP, TCAP, SCCP etc...), I know the 'disectors' are likely not existing, but it would be interresting to see what your plans are on the long run?

Bye Christophe

kaitoy commented 11 years ago

Hi dzonekl,

I have no concrete plan to support any protocols for now. At least, I think I should support common ones early, such as NDP, PPP, HTTP, etc. I don't think telecommunications protocols are common very much and so I will give low priorities to them. Basically, I hope users would add protocol supports themselves (and push them to my repository). I will document how to do that.

Thank you for your interest in Pcap4J, Kaito

dzonekl commented 11 years ago

Hello Kaito,

Thank you for your response. I understand your priorities regarding support for specific protocols. I would like to understand, what it take to add one protocol. I am looking forward to the "howto" document on this.

BTW, I have managed to run some of the tests examples of pcap4j, for example

Does this mean, the wrapper for libPcap works correctly? (I didn't install anything specific on MacOSX).

Also, I get a compile error on the files:

/pcap4j/src/main/java/org/pcap4j/core/PcapIpv4Address.java /pcap4j/src/main/java/org/pcap4j/core/PcapIpv6Address.java

The 'v' should be a captial letter, as the class name is : PcapIpV4Address, PcapIpV6Address

rgds Christophe

On Sun, May 5, 2013 at 8:44 AM, Kaito Yamada notifications@github.comwrote:

Hi dzonekl,

I have no concrete plan to support any protocols for now. At least, I think I should support common ones early, such as NDP, PPP, HTTP, etc. I don't think telecommunications protocols are common very much and so I will give low priorities to them. Basically, I hope users would add protocol supports themselves (and push them to my repository). I will document how to do that.

Thank you for your interest in Pcap4J, Kaito

— Reply to this email directly or view it on GitHubhttps://github.com/kaitoy/pcap4j/issues/9#issuecomment-17447561 .

Christophe Bouhier Christophe.bouhier@netxforge.com +31612962997 Clifforstraat 43-II 1051 GS, Amsterdam http://www.netxforge.com

kaitoy commented 11 years ago

Both PcapNetworkInterfaceTest and ReadPacketFile use some APIs of libpcap and so if it works it means the wrappers work well. As for compile error, thank you for your information. I've fixed the problem. I didn't notice the problem because my local repository is fine. When I pushed my local changes of class names to the remote repository, something bad happened and the file name changes did not go into the repository.

dzonekl commented 11 years ago

Thanks for the update. I have trouble pulling your change. I made a own branch (eclipsefy) to run pcap4j as an eclipse OSGI plugin, but pulling is complaining:

Christophes-MacBook-Pro:pcap4j Christophe$ git checkout eclipsefy error: The following untracked working tree files would be overwritten by checkout: src/main/java/org/pcap4j/core/PcapIpv4Address.java src/main/java/org/pcap4j/core/PcapIpv6Address.java target/.gitignore Please move or remove them before you can switch branches. Aborting I am not sure, what happend but the commit looks odd, with several commits manipulating them. I think you should revert back and try again, because this breaks somehow. (I didn't touch these files locally).

BTW, it would be nice if pcap4j could be offered as plugin, I can offer a patch for this if you think it's worth it. cheers. Christophe

On Fri, May 10, 2013 at 7:52 AM, Kaito Yamada notifications@github.comwrote:

PcapNetworkInterfaceTest uses some APIs of libpcap and so if it works it means the wrappers work well. As for compile error, thank you for your information. I've fixed the problem. I didn't notice the problem because my local repository is fine. When I pushed my local changes of class names to the remote repository, something bad happened and the file name changes did not go into the repository.

— Reply to this email directly or view it on GitHubhttps://github.com/kaitoy/pcap4j/issues/9#issuecomment-17705480 .

Christophe Bouhier Christophe.bouhier@netxforge.com +31612962997 Clifforstraat 43-II 1051 GS, Amsterdam http://www.netxforge.com

kaitoy commented 11 years ago

The pcap4j repository on Github seems to be fine. I can clone pcap4j from Github and build it successfully. I'm afraid your local repository or working tree has a problem. Please try to remove pcap4j you cloned and re-make a clone. As for plugin, I'm not familiar with it but if you will contribute something it would be great.

Kaito

kaitoy commented 11 years ago

Hi dzonekl,

I wrote how to add a protocol support: https://github.com/kaitoy/pcap4j/blob/master/www/HowToAddProtocolSupport.md I don't think this document is enough, but I'm not sure what should be added. If you give me feedback it helps.

Kaito

dzonekl commented 11 years ago

Hi, thanks. I will have a look and let you know. CHeers Christophe

On Sat, Jun 1, 2013 at 5:50 AM, Kaito Yamada notifications@github.comwrote:

Hi dzonekl,

I wrote how to add a protocol support: https://github.com/kaitoy/pcap4j/blob/master/www/HowToAddProtocolSupport.md I don't think this document is enough, but I'm not sure what should be added. If you give me feedback it helps.

Kaito

— Reply to this email directly or view it on GitHubhttps://github.com/kaitoy/pcap4j/issues/9#issuecomment-18783039 .

Christophe Bouhier Christophe.bouhier@netxforge.com +31612962997 Clifforstraat 43-II 1051 GS, Amsterdam http://www.netxforge.com