-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Detect count 5044 v2 #13902
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
base: main
Are you sure you want to change the base?
Detect count 5044 v2 #13902
Conversation
enip.command is not a keyword nor an alias
Ticket: 7211 Allows to count the number of elements, without matching on individual elements
Ticket: 5044
Information: QA ran without warnings. Pipeline = 27704 |
without matching to a specific value. | ||
|
||
The syntax is:: | ||
keyword: count [mode] value; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we document the usages such as all
, all1
, nb
etc that are also valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had missed the TODO
description in the PR, apologies.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious: this looks different than what is discussed here https://redmine.openinfosecfoundation.org/issues/5044#note-2
Would there be a follow up to support that, or a decision to approach the feature differently?
Good catch, So, this PR is implementing the ticket title, what is discussed further (repeated pattern) should be another ticket... |
Link to ticket: https://redmine.openinfosecfoundation.org/issues/
https://redmine.openinfosecfoundation.org/issues/5044
Describe changes:
count
option to multi-buffers, behaving like a keyword but syntax isemail.received: count <3;
instead ofemail.received; count: <3;
all
,all1
andnb
SV_BRANCH=OISF/suricata-verify#2634
Draft :
TODOs :