-
Notifications
You must be signed in to change notification settings - Fork 610
[Rule Tuning] Update PowerShell ES|QL Rules KEEP Condition #5391
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
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
|
| Esql.script_block_tmp, | ||
| powershell.file.script_block_text, | ||
| powershell.file.script_block_id, | ||
| powershell.file.script_block_entropy_bits, |
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.
what if the customer is using ps logs without the custom ingestion pipeline u added for entropy? the query will error ? if so I would suggest create one dedicated for ps with entropy enrichment to experiment with performance and adoption ?
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.
As long as customers are using the official integration to collect the logs (the one we support in the query), they will have the field, this is like any addition of fields done by integrations, as for performance, it is done in the ingest time, so there is no difference for the rules.
We can also do powershell.file.*, so we are sure to not fail the field verification
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 is available since Windows Integration v3.1.3, published a month ago)
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.
|
⛔️ Test failed Results
|
|
⛔️ Test failed Results
|
|
since you're doing a mass update on ESQL rule keep fields, could you also add |
Issues
Small tuning included that resolves #5263
Summary
Updates the KEEP condition of the PowerShell ES|QL rules so we start receiving telemetry on the new PowerShell fields.