crowdstrike: add support for NG-SIEM Correlation Detection events#19494
crowdstrike: add support for NG-SIEM Correlation Detection events#19494navnit-elastic wants to merge 6 commits into
Conversation
61da592 to
0db1c8d
Compare
✅ 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. |
| - append: | ||
| field: event.category | ||
| tag: append_event_category_threat | ||
| value: threat |
There was a problem hiding this comment.
@navnit-elastic this is the incorrect event.category can you adjust to event.category:alert
There was a problem hiding this comment.
@jamiehynds, Would below event categorization meet expectation?
event.kind: alert
event.category: threat
event.type: indicator
There was a problem hiding this comment.
@navnit-elastic event.kind: alert is correct but the category and type fields are incorrect as they're more suited to IOC's from Threat Intel feeds.
Can we set to:
event.kind: alert
event.type: info
There was a problem hiding this comment.
@jamiehynds, agreed. event.kind: alert set by default.yml, adjusted correlation_detection.yml to set only event.type: info.
| tag: append_event_category_threat | ||
| value: threat | ||
| - append: | ||
| field: event.type |
There was a problem hiding this comment.
can drop this as we're adjusting the event.category to alert
0db1c8d to
06d00e9
Compare
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
🚀 Benchmarks reportPackage
|
| Data stream | Previous EPS | New EPS | Diff (%) | Result |
|---|---|---|---|---|
falcon |
6802.72 | 3133.81 | -3668.91 (-53.93%) | 💔 |
falcon |
6802.72 | 5524.86 | -1277.86 (-18.78%) | 💔 |
To see the full report comment with /test benchmark fullreport
|
Is the |
@jamiehynds, The |
|
✅ All changelog entries have the correct PR link. |
TL;DRBuildkite build 44534 failed before tests because the repository post-checkout hook could not merge commit Remediation
Investigation detailsRoot CauseThis is a merge-conflict/configuration failure during Buildkite checkout, not a package test failure. In build 44534, the post-checkout hook tried to merge the PR commit into At 2 - version: "3.24.0"
3 changes:
4 - description: Make max_executions configurable for all CEL data streams to allow high-volume environments to complete pagination within a single interval.
5 type: enhancement
6 link: https://github.com/elastic/integrations/pull/19530The failed PR commit The current PR head Evidence
Verification
What is this? | From workflow: PR Buildkite Detective Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not. |
💚 Build Succeeded
History
|
efd6
left a comment
There was a problem hiding this comment.
LGTM, but I'd like to know the answer. Also, please wait for @jamiehynds.
Proposed commit message
Note
Acceptance criteria
event.severity,event.start,event.end, andevent.durationare populated -event.severityis derived fromseverity_nameto align with other CrowdStrike alert types.host.name,user.name, andrule.nameare populated for NG-SIEM events -host.nameis derived fromhost_names[],user.nameis derived fromuser_names[](fall back tousers[].user_name), andrule.nameis derived fromdisplay_name.related.hosts,related.user, andrelated.ipare populatedrule.idis populated fromcorrelation_rule_idcorrelation_rule_*fields land undercrowdstrike.alert.*- this is not required as the pipeline renames thejson.*is removed at the end of pipeline processingjson.*object tocrowdstrike.alert.*fields.ymlfor any newcrowdstrike.alert.*fields introducedChecklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots