Feature #5466
closeddetect: allow alert-then-pass logic
Description
Currently pass acts as a noalert rule that stops further alerting.
Some usecases have been identified in which ppl want a "alert then pass" in a single rule. Currently they are forced to express this in 2 rules, an alert rule and a pass rule, where the action order and or priorities needs to be setup such that the alert rule is evaluated first.
We do support the following: alert .... (bypass; ...)
.
I think we could extend the config
rule keyword for this.
E.g. something like:alert ... (config:logging disable, type alert, scope flow;)
The behavior would still need to log the current alert, so it needs a bit of thought on how to express this.
Updated by Jason Ish almost 2 years ago
One thought, a companion keyword to noalert
: alert
.
pass ftp a b -> c d (msg:"LOG This as a reason why the flow passed"; alert; ...;)
Updated by Shivani Bhardwaj almost 2 years ago
I like Jason's idea. Other thoughts:
1. config:action pass, type alert, scope flow
Not sure if adding more than logging subsystem to config would be too much pain though.
2. new keyword to define next action post match. action:pass
which sets the rule's action to the defined action after executing the current defined action on the rule. (perhaps some other name to avoid confusion with the logs)
Updated by Juliana Fajardini Reichow over 1 year ago
As a first solution, think of achieving this as a configuration option to log `PASS` rules to the alert eve type.
Updated by Juliana Fajardini Reichow over 1 year ago
- Target version changed from TBD to 7.0.0-rc2
Updated by Juliana Fajardini Reichow over 1 year ago
- Target version changed from 7.0.0-rc2 to 8.0.0-beta1
Updated by Victor Julien 10 months ago
- Status changed from New to Assigned
- Assignee changed from OISF Dev to Victor Julien
- Label Needs backport to 7.0 added
Updated by Victor Julien 10 months ago
- Status changed from Assigned to In Review
Updated by Juliana Fajardini Reichow 10 months ago
- Related to Documentation #6685: userguide: explain noalert keyword added
Updated by Juliana Fajardini Reichow 6 months ago
Related RFC PR: https://github.com/OISF/suricata/pull/10157
Updated by Victor Julien 5 months ago
- Status changed from In Review to Resolved