-
Notifications
You must be signed in to change notification settings - Fork 611
[Tuning] Diverse Rules Tuning #5482
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?
Conversation
Samirbous
commented
Dec 17, 2025
- Bash Shell Profile Modification - remove rename from event.action and uses file.name for better matches).
- Web Shell Detection: Script Process Child of Common Web Processes - extra tuning to reduce FPs
Rule: Tuning - GuidelinesThese guidelines serve as a reminder set of considerations when tuning an existing rule. Documentation and Context
Rule Metadata Checks
Testing and Validation
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
| severity = "high" | ||
| tags = [ | ||
| "OS: Windows", | ||
| "Data Source: Elastic Endgame", |
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.
| severity = "low" | ||
| tags = [ | ||
| "OS: Windows", | ||
| "Data Source: Elastic Endgame", |
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.
| @@ -1,8 +1,8 @@ | |||
| [metadata] | |||
| creation_date = "2023/10/16" | |||
| integration = ["problemchild", "endpoint"] | |||
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 this include windows?
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.
it will just add unnecessary noise, the rule require fixes at the ML level, this tuning purpose is to reduce the flooding.
| /* | ||
| This rule does not exclude local IP ranges by default. To exclude these, use the following exclusion statement: | ||
| cidrmatch(destination.ip, "10.0.0.0/8", "169.254.0.0/16", "172.16.0.0/12", "192.0.0.0/24", "192.0.0.0/29", "192.0.0.8/32", "192.0.0.9/32", | ||
| not cidrmatch(destination.ip, "127.0.0.0/8", "::1","FE80::/10", "FF00::/8", "10.0.0.0/8", "169.254.0.0/16", "172.16.0.0/12", "192.0.0.0/24", "192.0.0.0/29", "192.0.0.8/32", "192.0.0.9/32", |
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.
No longer detecting internal c2 right?
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.
99% of the noise is from internal apps, difficult to spot internal C2 in all the noise using this logic.
rules/cross-platform/persistence_shell_profile_modification.toml
Outdated
Show resolved
Hide resolved
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|