[Security solution] Stops page from crashing when there is a fields error in the stackBy component

[stephmilovic]

Summary

Resolves #149540

We've been seeing SDHs reporting a stack overflow error that breaks the page.

The error points to a valid problem, but we wanted to improve the Alerts page UX to prevent the app to crash in these situations, making the application more resilient and showing some meaningful message to the customer when this happens.

Testing

Start by running local es from a snapshot and local kibana. Unless you want to edit some paths in scripts the best way to do this is to run kibana with a base path and defining that base path to be kbn by adding this line to your kibana.dev.yml:

server.basePath: '/kbn'

@e40pud created a super helpful script for generating indices with huge amounts of fields: https://github.com/elastic/kibana/tree/main/x-pack/plugins/security_solution/scripts/mappings

Using Huge Indices tool, cd into the directory x-pack/plugins/security_solution/scripts/mappings and run the following commands:

1. Creates a huge field mapping. Uses 371k fields because thats what our most recent SDH client had. Defines the index as filebeat- so the index will become part of our Security Data View without changes to the advanced settings

   node mappings_generator.js --fieldsCount=371675 --indexCount=1 --indexPrefix='filebeat-' --unmappedRate=.2 --buckets=10 --outputDirectory='test_unmapped'
   

2. Load the mappings into an index in your environment

   node mappings_loader.js --mappings-dir='test_unmapped' --es-url=http://elastic:changeme@localhost:9200 --kibana-url=http://elastic:changeme@localhost:5601/kbn/ap
   

Now check the overview page. You should see our new error coming up. If you check out main, you will see the crash.

Before

After

[elasticmachine]

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

[elasticmachine]

Pinging @elastic/security-solution (Team: SecuritySolution)

[stephmilovic]

useCallback so this does not run right away, just when called

[stephmilovic]

@paulewing what do you think of this message? Looks like:

[janmonschke]

Thanks for adding this fix, @stephmilovic 🎉
Some tests might need updating with your changes but otherwise looks good to me.

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: a683287

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	13.0MB	13.0MB	+1.5KB

History

• 💔 Build #166463 failed fcdba45

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