Skip to content

[8.10] [Security Solution][Endpoint] Fixes kibana crash when going back to policy details page (#164329)#164555

Merged
kibanamachine merged 2 commits intoelastic:8.10from
kibanamachine:backport/8.10/pr-164329
Aug 23, 2023
Merged

[8.10] [Security Solution][Endpoint] Fixes kibana crash when going back to policy details page (#164329)#164555
kibanamachine merged 2 commits intoelastic:8.10from
kibanamachine:backport/8.10/pr-164329

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.10:

Questions ?

Please refer to the Backport tool documentation

…olicy details page (elastic#164329)

Fixes: elastic#164295
Fixes: elastic#164190

## Summary
There was a Kibana crash when using back link buttons to go back to the
policy details page from Trusted Apps / Event Filters / Host Isolation
exceptions / Blocklists page. Route state can be null depending on the
previous action so it has to be conditional.

The error occurs when it tries to access the `backLink` prop on
`routeState` when it's not defined.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit fb79e3e)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Aug 23, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) August 23, 2023 10:50
@dasansol92
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Aug 23, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Security Investigations Cypress Tests #2 / timeline data providers persists timeline when data provider is updated by dragging a field from data grid persists timeline when data provider is updated by dragging a field from data grid

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 15.7MB 15.7MB +15.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @dasansol92

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