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

Add monitoring capability #2588

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Add monitoring capability #2588

merged 1 commit into from
Oct 14, 2024

Conversation

IvanNardi
Copy link
Collaborator

Allow nDPI to process the entire flows and not only the first N packets. Usefull when the application is interested in some metadata spanning the entire life of the session.

As initial step, only STUN flows can be put in monitoring.

See doc/monitoring.md for further details.

This feature is disabled by default.

Close #2583

@IvanNardi IvanNardi force-pushed the monitoring branch 3 times, most recently from debd35e to c2ff178 Compare October 13, 2024 18:25
doc/monitoring.md Outdated Show resolved Hide resolved
doc/monitoring.md Outdated Show resolved Hide resolved
doc/monitoring.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@utoni utoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for that feature Ivan. That is something that Cybersecurity folks need!

Do you think that it would make sense to also provide an API in which makes it possible to dynamically add a certain flow to the "special observation treatment" and dissect literally all packets of it?

@IvanNardi
Copy link
Collaborator Author

Do you think that it would make sense to also provide an API in which makes it possible to dynamically add a certain flow to the "special observation treatment" and dissect literally all packets of it?

@utoni, could you elaborate please? Do you mean something like: the application want to put on monitoring only THIS specific STUN flow and not all of them?

@utoni
Copy link
Collaborator

utoni commented Oct 14, 2024

Exactly, but I would like to use that specific behavior for other protocols e.g. suspicious TLS or BitTorrent flows.

@IvanNardi
Copy link
Collaborator Author

Exactly, but I would like to use that specific behavior for other protocols e.g. suspicious TLS or BitTorrent flows.

Right now the code has been updated only for STUN; to add support for monitoring to other protocols you need to update their dissectors code.

About the specific flow. The configuration is per protocols, so all the STUN flows or none of it, but the final decision is always of the application itself: the application can simply stop invoking nDPI for the "uninteresting" STUN flows.

That is the current state; we can always improve and extend it

Allow nDPI to process the entire flows and not only the first N packets.
Usefull when the application is interested in some metadata spanning the
entire life of the session.

As initial step, only STUN flows can be put in monitoring.

See `doc/monitoring.md` for further details.

This feature is disabled by default.

Close ntop#2583
Copy link

@IvanNardi IvanNardi merged commit 521d0ca into ntop:dev Oct 14, 2024
37 checks passed
@IvanNardi IvanNardi deleted the monitoring branch October 14, 2024 16:05
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.

Enhance STUN implementation
3 participants