Skip to content

Add Webex STUN sub-classification via SOFTWARE attribute#3194

Merged
IvanNardi merged 1 commit into
ntop:devfrom
cmonroe:webex
Jul 14, 2026
Merged

Add Webex STUN sub-classification via SOFTWARE attribute#3194
IvanNardi merged 1 commit into
ntop:devfrom
cmonroe:webex

Conversation

@cmonroe

@cmonroe cmonroe commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Please sign (check) the below before submitting the Pull Request:

Link to the related issue:

Describe changes:
Webex call media uses ICE/STUN. The STUN control flows were reported as STUN/Network because the STUN dissector had no Webex sub-classification hook even though the media servers already fall in the known Webex IP ranges.

Webex (formerly Cisco Spark) ICE agents advertise the STUN SOFTWARE attribute (0x8022) with the value "Cisco-Spark". Parse it in is_stun() and set the app protocol to the existing Webex protocol so the flow classifies as STUN.Webex (VoIP) with DPI confidence.

Add a minimal test capture and its expected result.

Webex call media uses ICE/STUN. The STUN control flows were
reported as STUN/Network because the STUN dissector had no Webex
sub-classification hook even though the media servers already fall
in the known Webex IP ranges.

Webex (formerly Cisco Spark) ICE agents advertise the STUN SOFTWARE
attribute (0x8022) with the value "Cisco-Spark". Parse it in
is_stun() and set the app protocol to the existing Webex protocol so
the flow classifies as STUN.Webex (VoIP) with DPI confidence.

Add a minimal test capture and its expected result.

Signed-off-by: Chad Monroe <chad@monroe.io>
@sonarqubecloud

Copy link
Copy Markdown

@IvanNardi IvanNardi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you

@IvanNardi IvanNardi merged commit 1ac558b into ntop:dev Jul 14, 2026
28 checks passed
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