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

UDP support #62

Closed
raboof opened this issue Mar 27, 2016 · 4 comments · Fixed by #199
Closed

UDP support #62

raboof opened this issue Mar 27, 2016 · 4 comments · Fixed by #199

Comments

@raboof
Copy link
Owner

raboof commented Mar 27, 2016

No description provided.

@pavan02
Copy link
Contributor

pavan02 commented May 3, 2018

Is this still open ? I can see #10 open as well..

If not its implemented yet, what does the below callback functions do?

https://github.com/raboof/nethogs/blob/master/src/main.cpp#L241

dp_addcb(newhandle, dp_packet_ip, process_ip);
dp_addcb(newhandle, dp_packet_ip6, process_ip6);
dp_addcb(newhandle, dp_packet_tcp, process_tcp);
dp_addcb(newhandle, dp_packet_udp, process_udp);

@raboof
Copy link
Owner Author

raboof commented May 3, 2018

Nethogs used to capture UDP traffic and show all UDP traffic as 'unknown UDP'. I think I stopped showing that because it wasn't very useful anyway, and I'm not sure I could reliably determine whether the packets were incoming or outgoing. Actually reliably associating them with processes I'm not sure is possible.

So yes, this ticket is open, but I'm not sure if there's any useful information we can reasonably show for UDP traffic. What do you think?

@pavan02
Copy link
Contributor

pavan02 commented May 6, 2018

Hmm...I still trying to figure out how nethogs work with TCP....any pointers would be of great help.
What do mean by reliably ? What's the difficulty/tricky part with UDP traffic?

Total data usage like in this ticket #109 could be useful...

BTW what about #10 ? Is it already implemented ?

@raboof
Copy link
Owner Author

raboof commented May 6, 2018

Wrt #10, I'm not entirely sure

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

Successfully merging a pull request may close this issue.

2 participants