M365 Defender integration - fix on process.command_line - #8872
Conversation
kcreddy
left a comment
There was a problem hiding this comment.
Can you add an entry into changelog.yml like this: https://github.com/elastic/integrations/blob/main/packages/m365_defender/changelog.yml#L12-L16. Since this is a bugfix, the version here would be 2.5.1
Can you also update manifest.yml here with version 2.5.1?
|
@kcreddy done. |
|
/test |
After the change to the painless script, your change is valid according to this block of code: |
Co-authored-by: Krishna Chaitanya Reddy Burri <krishnachaitanyareddy.burri@elastic.co>
🚀 Benchmarks reportPackage
|
| Data stream | Previous EPS | New EPS | Diff (%) | Result |
|---|---|---|---|---|
event |
723.07 | 578.03 | -145.04 (-20.06%) | 💔 |
To see the full report comment with /test benchmark fullreport
|
Package m365_defender - 2.5.1 containing this change is available at https://epr.elastic.co/search?package=m365_defender |
process.command_linewas overwritten byprocess.parent.command_linedue to a typo in the script parsing the parent.command lineProposed commit message
fix process.command_line overwritten by process.parent.command_line
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots