Skip to content

[9.0] [Security Solution] Avoid showing rules update confirmation modal on basic license (#214340)#214385

Merged
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-214340
Mar 13, 2025
Merged

[9.0] [Security Solution] Avoid showing rules update confirmation modal on basic license (#214340)#214385
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-214340

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

…basic license (elastic#214340)

**Resolves:** elastic#214302

## Summary

This PR prevents showing rule upgrade confirmation modal on lower licenses where prebuilt rules customization is not allowed.

## Details

Users may see a rules upgrade confirmation modal when trying to upgrade prebuilt rules even if prebuilt rules customization is disabled due to insufficient license. It happens due to improper response from `upgrade/_perform` which doesn't respect `pick_version`. It's expected rule upgrade goes smoothly when `pick_version` is one of `BASE`, `CURRENT` or `TARGET`.

The fix makes sure dry run request isn't fired and a prebuilt rules upgrade confirmation modal isn't shown when running with insufficient for prebuilt rules customization license.

There is a [ticket](elastic#214338) to address this issue in the API endpoint.

(cherry picked from commit d782b71)
@kibanamachine kibanamachine merged commit 2ae824c into elastic:9.0 Mar 13, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #12 / get_column_renderer should render a zeek row data if event.category is network_traffic

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 +52.0B

cc @maximpn

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