Project

General

Profile

Actions

Feature #3952

closed
SB JI

protocols: implement mDNS

Feature #3952: protocols: implement mDNS

Added by Shivani Bhardwaj over 5 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:
Protocol

Description

Support for mDNS protocol in Suricata in Rust.


Related issues 6 (3 open3 closed)

Related to Suricata - Task #4151: Research: New protocol supportNewCommunity TicketActions
Related to Suricata - Feature #5773: doh: support DNS over HTTPS (DoH)ClosedPhilippe AntoineActions
Related to Suricata - Feature #6453: Support DNS over TLSNewOISF DevActions
Related to Suricata - Bug #6281: dns: structure of query differs between "alert" and "dns" event typesClosedJason IshActions
Blocks Suricata - Task #7118: tracking: add support for new protocolsAssignedVictor JulienActions
Blocks Suricata - Story #7119: protocols: protocol additionsClosedVictor JulienActions

VJ Updated by Victor Julien over 5 years ago Actions #1

  • Label Protocol added

PA Updated by Philippe Antoine over 3 years ago Actions #2

mDNS is just DNS on another port, right ?

PA Updated by Philippe Antoine over 2 years ago Actions #3

  • Related to Task #4151: Research: New protocol support added

VJ Updated by Victor Julien over 2 years ago Actions #4

  • Status changed from New to Assigned
  • Assignee set to Jason Ish
  • Target version set to TBD

VJ Updated by Victor Julien over 2 years ago Actions #5

  • Related to Feature #5773: doh: support DNS over HTTPS (DoH) added

VJ Updated by Victor Julien over 2 years ago Actions #6

VJ Updated by Victor Julien over 2 years ago Actions #7

  • Target version changed from TBD to 8.0.0-beta1

VJ Updated by Victor Julien almost 2 years ago Actions #8

I've looked a bit into this, and so far it seems that it's indeed just dns over 5353 so far. Adding 5353 to the UDP DNS ports in the yaml is enough to get logging going.

To me the main question now is: is this enough, or should we distinguish between dns and mdns? It's fairly trivial to reregister the dns parsers as mdns, so they get their own log type. This would require dns keywords to be updated as well.

On the other hand, we can also just annotate regular dns records as mdns by looking at the ports and/or dest address.

VJ Updated by Victor Julien almost 2 years ago Actions #9

One thing to consider if that existing rules won't expect mdns/5353 to match app proto dns.

VJ Updated by Victor Julien almost 2 years ago Actions #10

Related: there are several other DNS-like protocols. Netbios name service uses DNS structure but with a special name encoding. There is link local mnds too, using port 5355.

JI Updated by Jason Ish almost 2 years ago Actions #11

  • Related to Bug #6281: dns: structure of query differs between "alert" and "dns" event types added

JI Updated by Jason Ish almost 2 years ago Actions #12

Related to #6281. MDNS actually uses the ability to have multiple queries in the request which is one of the logging issues #6281 addresses.

I wonder if its safe enough to identify DNS on port 5353 as mdns? If we need a further restriction, multicast and port 5353.

VJ Updated by Victor Julien almost 2 years ago Actions #13

Conclusion after some discussion:

dns, doh, dns over tls should be grouped together as app proto dns
mdns and it's variants should be separate

VJ Updated by Victor Julien almost 2 years ago Actions #14

  • Blocks Task #7118: tracking: add support for new protocols added

VJ Updated by Victor Julien almost 2 years ago Actions #15

  • Blocks Story #7119: protocols: protocol additions added

VJ Updated by Victor Julien about 1 year ago Actions #16

  • Target version changed from 8.0.0-beta1 to 8.0.0-rc1

If we get to it, we can add some support for this post beta1, but disabled by default.

SB Updated by Shivani Bhardwaj about 1 year ago Actions #17

  • Subject changed from mDNS protocol implementation to protocols: implement mDNS

JI Updated by Jason Ish 11 months ago Actions #18

  • Status changed from Assigned to In Progress

JI Updated by Jason Ish 11 months ago Actions #19

  • Status changed from In Progress to In Review

JI Updated by Jason Ish 11 months ago Actions #20

  • Status changed from In Review to Closed

Merged.

Actions

Also available in: PDF Atom