Skip to content

[9.3] [Security Solution][Detection Engine] Fix alert count so max alerts warning shows correctly (#259199)#259805

Merged
kibanamachine merged 3 commits intoelastic:9.3from
kibanamachine:backport/9.3/pr-259199
Mar 27, 2026
Merged

[9.3] [Security Solution][Detection Engine] Fix alert count so max alerts warning shows correctly (#259199)#259805
kibanamachine merged 3 commits intoelastic:9.3from
kibanamachine:backport/9.3/pr-259199

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

…arning shows correctly (elastic#259199)

Fixes elastic#259169

`createEventSignal` has a bug where it returns incorrect summary results
for pages of source docs that matched no indicators. The calling code
expects `createEventSignal` to return results pertaining only to the
current page, but if no indicators are matched or an error is
encountered, the function instead returns `currentResults` i.e. the sum
of results from all prior pages. The effect is that each time a page
matches no indicators the alert count we track in `createThreatSignals`
_doubles_ because we add `currentResults` to itself.

(cherry picked from commit 22c93a4)
@kibanamachine kibanamachine merged commit 519bd35 into elastic:9.3 Mar 27, 2026
14 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @marshallmain

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