Skip to content

Commit

Permalink
[Security Solution] Update insight depth limit to 1 (elastic#161901)
Browse files Browse the repository at this point in the history
## Summary

Before elastic#160691 was merged, the
filter builder depth of 1 was the same as 0, and so I mistakenly had
this value set to 2. With it set to 2, it's possible to create a filter
with top level AND and 2 levels of nesting that will cause the designed
for only 1 level of nesting data provider code to throw a run time
error. Setting this to the now working level of 1 prevents that.

(cherry picked from commit 97705ff)
  • Loading branch information
kqualters-elastic committed Jul 13, 2023
1 parent 85a2e1b commit 25c3fed
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,7 @@ const InsightEditorComponent = ({
filters={filtersStub}
onChange={onChange}
dataView={dataView}
maxDepth={2}
maxDepth={1}
/>
) : (
<></>
Expand Down

0 comments on commit 25c3fed

Please sign in to comment.