Skip to content

[Defender Endpoint] fix null ref for description description - #12369

Merged
kcreddy merged 0 commit into
elastic:mainfrom
SimonKoetting:defender_endpoint_description_failsafe
Jan 20, 2025
Merged

[Defender Endpoint] fix null ref for description description#12369
kcreddy merged 0 commit into
elastic:mainfrom
SimonKoetting:defender_endpoint_description_failsafe

Conversation

@SimonKoetting

Copy link
Copy Markdown
Contributor

There can be events without a description, which leads the if condition to fail with:
cannot access method/field [length] from a null def reference

@SimonKoetting
SimonKoetting requested a review from a team as a code owner January 16, 2025 13:20
@andrewkroh andrewkroh added bugfix Pull request that fixes a bug issue Integration:microsoft_defender_endpoint Microsoft Defender for Endpoint Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jan 16, 2025
@elasticmachine

Copy link
Copy Markdown

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

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

Comment thread packages/microsoft_defender_endpoint/changelog.yml Outdated
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

  • 💚 Build #20517 succeeded aa86b230fb41f084d6a8557fc3fe5edc83eaff65
@SimonKoetting
SimonKoetting requested a review from kcreddy January 20, 2025 10:19
@kcreddy

kcreddy commented Jan 20, 2025

Copy link
Copy Markdown
Contributor

@SimonKoetting, let me know if you don't have access to merge the PR.

@SimonKoetting

Copy link
Copy Markdown
Contributor Author

@kcreddy thanks, yes, thats the case :D would be nice if you could merge it. Thanks!

@kcreddy
kcreddy merged commit a4e7582 into elastic:main Jan 20, 2025
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
…#12369)

There can be events without a description, which leads the if condition to fail with:
cannot access method/field [length] from a null def reference
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
…#12369)

There can be events without a description, which leads the if condition to fail with:
cannot access method/field [length] from a null def reference
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:microsoft_defender_endpoint Microsoft Defender for Endpoint Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

4 participants