Skip to content

[Crowdstrike] - Handle nested File object in FDR DataEgressEnriched events#18411

Merged
ShourieG merged 3 commits intoelastic:mainfrom
ShourieG:bugfix/crowdstrike_fdr
Apr 16, 2026
Merged

[Crowdstrike] - Handle nested File object in FDR DataEgressEnriched events#18411
ShourieG merged 3 commits intoelastic:mainfrom
ShourieG:bugfix/crowdstrike_fdr

Conversation

@ShourieG
Copy link
Copy Markdown
Contributor

Type of change

  • Bug

Proposed commit message

crowdstrike: handle nested File object in FDR DataEgressEnriched events

DataEgressEnriched events send crowdstrike.File as a nested JSON object
instead of a keyword, causing document rejections. Extract sub-fields to
ECS file.* and crowdstrike.FileType.Type.*, then remove the Map before
indexing. Events where File is already a string are unaffected.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

DataEgressEnriched events send crowdstrike.File as a nested JSON object
instead of a keyword, causing document rejections. Extract sub-fields to
ECS file.* and crowdstrike.FileType.Type.*, then remove the Map before
indexing. Events where File is already a string are unaffected.
@ShourieG ShourieG self-assigned this Apr 15, 2026
@ShourieG ShourieG requested a review from a team as a code owner April 15, 2026 12:23
@ShourieG ShourieG added Integration:crowdstrike CrowdStrike bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:Security-Cloud Services Security Data Experience - Cloud Services team [elastic/cloud-services] labels Apr 15, 2026
@elasticmachine
Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Copy link
Copy Markdown
Contributor

@kcreddy kcreddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
@navnit-elastic, please make a note of this new event type addition.

Comment thread packages/crowdstrike/data_stream/fdr/elasticsearch/ingest_pipeline/default.yml Outdated
@kcreddy
Copy link
Copy Markdown
Contributor

kcreddy commented Apr 15, 2026

@ShourieG, is the label Team:Security-Cloud Services auto-added?

@ShourieG
Copy link
Copy Markdown
Contributor Author

ShourieG commented Apr 16, 2026

@ShourieG, is the label Team:Security-Cloud Services auto-added?

No I added it manually, this is labeled as Security Data Experience - Cloud Services which aligns with the current name change sde-service-integrations. At least this was my interpretation of it.

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @ShourieG

@ShourieG ShourieG merged commit ca689c6 into elastic:main Apr 16, 2026
10 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package crowdstrike - 3.14.1 containing this change is available at https://epr.elastic.co/package/crowdstrike/3.14.1/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue Integration:crowdstrike CrowdStrike Team:Security-Cloud Services Security Data Experience - Cloud Services team [elastic/cloud-services] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants