Skip to content

[9.3] [Security Solution] fix potential issues with loading data views in detection engine (#256996)#258236

Merged
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-256996
Mar 17, 2026
Merged

[9.3] [Security Solution] fix potential issues with loading data views in detection engine (#256996)#258236
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-256996

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

…etection engine (elastic#256996)

## Summary

A recent SDH made us realize that we have a potential issue in our code,
when the platform dataview service throws errors.
This previous [PR](elastic#256983) fixes
the issue in the @elastic/security-threat-hunting-investigations
codebase and we want to merge it for `9.2.7`.

This PR applies similar changes to the code owned by the
@elastic/security-detection-engine team:
- wrap the `dataview.get` calls within a `try/catch` and show a toast if
an error happens

This will facilitate debugging if we ever need. (more info in the PR
mentioned above).

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit 243e138)
@kibanamachine kibanamachine merged commit 6c5dc90 into elastic:9.3 Mar 17, 2026
19 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 10.8MB 10.8MB +372.0B

cc @PhilippeOberti

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants