Skip to content

[9.0] [Entity Analytics] [bug-fix] Do not abort risk score search request (#219858)#220209

Merged
kibanamachine merged 3 commits intoelastic:9.0from
kibanamachine:backport/9.0/pr-219858
May 23, 2025
Merged

[9.0] [Entity Analytics] [bug-fix] Do not abort risk score search request (#219858)#220209
kibanamachine merged 3 commits intoelastic:9.0from
kibanamachine:backport/9.0/pr-219858

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…lastic#219858)

## Summary

Closes elastic#219490

Fix an error toast appearing when changing an entities asset criticality
from the entity flyout. See video on issue above for more detail.

We were seeing the risk score request be aborted and this was creating
the error toast.

- Do not toast if an abort error is thrown
- Do not abort the request unless the component receives `skip:true`

### Test Steps

1. Use the security document generator `yarn start entity-store` command
to load entities and enable the risk engine
2. Enable the entity store
3. from the entities table open the entity flyout
4. Assign/change asset criticality
5. observe the error is not present

### Why did the bug happen?
The risk score component stops rendering because the conditions for
aborting and searching differ. So we aborted the HTTP call but didn't
search again.

I believe this PR elastic#201810
introduced the bug

---------

Co-authored-by: machadoum <pablo.nevesmachado@elastic.co>
(cherry picked from commit 64d20bb)
@hop-dev
Copy link
Contributor

hop-dev commented May 23, 2025

@elasticmachine merge upstream

@kibanamachine kibanamachine merged commit 9e1ce93 into elastic:9.0 May 23, 2025
9 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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 8.8MB 8.8MB +53.0B

History

cc @hop-dev

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.

4 participants