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

Using Node_pcap library to sniff the interface live and create pcap file #213

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

harrydevnull
Copy link

@harrydevnull harrydevnull commented Jul 6, 2016

Using Node_pcap library to sniff the interface live and create pcap file.
currently there is no way in the existing library to create a pcap file. I have created a new method on the add on. A new parameter (total no of packets to be captured ) has been added. Even though code looks more or less the same; i felt it was better to keep to to different method and different js file for code maintainability. An example has been added as pcap_dump_test.js.

Please let me know this is pull request is good enough for getting merged.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 61.477% when pulling b79e78a on harrydevnull:master into d920204 on mranney:master.

1 similar comment
@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage decreased (-0.6%) to 61.477% when pulling b79e78a on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage increased (+2.6%) to 64.677% when pulling 0041ff3 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage increased (+2.6%) to 64.677% when pulling 0041ff3 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage increased (+2.8%) to 64.8% when pulling 49b39d7 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 8, 2016

Coverage Status

Coverage increased (+2.8%) to 64.862% when pulling 2f68932 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 8, 2016

Coverage Status

Coverage increased (+2.8%) to 64.862% when pulling 98e9a51 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 9, 2016

Coverage Status

Coverage increased (+2.8%) to 64.862% when pulling 829f964 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 9, 2016

Coverage Status

Coverage increased (+2.8%) to 64.862% when pulling fefafbb on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Jul 28, 2016

Coverage Status

Coverage increased (+2.8%) to 64.862% when pulling 3101eb4 on harrydevnull:master into d920204 on mranney:master.

@coveralls
Copy link

coveralls commented Aug 1, 2016

Coverage Status

Coverage increased (+2.7%) to 64.782% when pulling ed491cb on harrydevnull:master into d920204 on mranney:master.

under the hood it makes use of the libpcap's capability to create pcap
the add-on makes use of the Async worker
@coveralls
Copy link

coveralls commented Sep 30, 2016

Coverage Status

Coverage increased (+2.7%) to 64.782% when pulling f6585fc on harrydevnull:master into d920204 on mranney:master.

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 this pull request may close these issues.

2 participants