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

evaluating pcap4j #9

Closed
dzonekl opened this issue May 3, 2013 · 7 comments
Closed

evaluating pcap4j #9

dzonekl opened this issue May 3, 2013 · 7 comments

Comments

@dzonekl
Copy link

dzonekl commented May 3, 2013

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
Copy link
Owner

kaitoy commented May 5, 2013

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
Copy link
Author

dzonekl commented May 7, 2013

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

  • PcapNetworkInterfaceTest
  • ReadPacketFile.

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//issues/9#issuecomment-17447561
.

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

@kaitoy
Copy link
Owner

kaitoy commented May 10, 2013

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
Copy link
Author

dzonekl commented May 11, 2013

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//issues/9#issuecomment-17705480
.

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

@kaitoy
Copy link
Owner

kaitoy commented May 14, 2013

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
Copy link
Owner

kaitoy commented Jun 1, 2013

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
Copy link
Author

dzonekl commented Jun 2, 2013

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//issues/9#issuecomment-18783039
.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants