Skip to content

Antoineaboufayssal/dataset match subdomain/v2.1 - #15605

Closed
catenacyber wants to merge 3 commits into
OISF:mainfrom
catenacyber:antoineaboufayssal/dataset-match-subdomain/v2.1
Closed

Antoineaboufayssal/dataset match subdomain/v2.1#15605
catenacyber wants to merge 3 commits into
OISF:mainfrom
catenacyber:antoineaboufayssal/dataset-match-subdomain/v2.1

Conversation

@catenacyber

Copy link
Copy Markdown
Contributor

#15073 + one commit fot green CI

antoineaboufayssal and others added 3 commits June 11, 2026 10:57
Ticket: 8385

Add a new match subdomain option that enables blocking a domain and all its subdomains using datasets.
@catenacyber catenacyber mentioned this pull request Jun 11, 2026
4 tasks
.. container:: example-rule

reject dns any any -> any any (msg:"Blocked domain"; dns.query; dotprefix; dataset:isset,blocked-domains, type string, match subdomain, load blocked-domains.lst; sid:8000003; rev:1;)
alert dns any any -> any any (msg:"Blocked domain"; dns.query; dotprefix; dataset:isset,blocked-domains, type string, match subdomain, load blocked-domains.lst; sid:8000003; rev:1;)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Can we have a requires keyword that works for reject ?

Because I got locally Error: detect-parse: Libnet 1.1.x is required for action "reject" but is not compiled into Suricata [SigParseActionRejectValidate:detect-parse.c:1523]

@victorjulien

Copy link
Copy Markdown
Member

Can you do a PR with just bdeda40 ?

Then I'll merge #15601 and get that new PR processed quickly as well.

@catenacyber

Copy link
Copy Markdown
Contributor Author

Can you do a PR with just bdeda40 ?

Then I'll merge #15601 and get that new PR processed quickly as well.

Done in #15606

Side note : That does not change my local reject problem

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants