Update influencer fields of Security ML integrations#20335
Update influencer fields of Security ML integrations#20335sodhikirti07 wants to merge 3 commits into
Conversation
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
|
Pinging @elastic/sec-applied-ml (Team:Security-Applied ML) |
|
✅ All changelog entries have the correct PR link. |
💚 Build Succeeded
|
| field: user.id | ||
| script: | ||
| source: "return doc.containsKey('user.name') && !doc['user.name'].isEmpty() ? doc['user.name'].value : ''" | ||
| lang: painless |
There was a problem hiding this comment.
Can we use missing_bucket: true for these instead?
| field: network.transport | ||
| 'process.executable': | ||
| terms: | ||
| field: process.executable |
There was a problem hiding this comment.
One other thing I picked up, if you are using Network Packet Capture as the data source, do you need to enable the monitor_processes setting in the integration policy?
https://github.com/elastic/integrations/blob/63384ae307ea445e86c064c37c84eebf24815e97/packages/network_traffic/_dev/build/docs/README.md#monitor_processes
|
Tick the box to add this pull request to the merge queue (same as
|
Proposed commit message
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Using ITP
Related issues
Screenshots