Skip to content

[8.x] [Security Solution] Avoid showing rules update confirmation modal on basic license (#214340)#214384

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-214340
Mar 13, 2025
Merged

[8.x] [Security Solution] Avoid showing rules update confirmation modal on basic license (#214340)#214384
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-214340

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

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 added the backport This PR is a backport of another PR label Mar 13, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 13, 2025 13:53
@kibanamachine kibanamachine merged commit 95e28b5 into elastic:8.x Mar 13, 2025
11 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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 9.1MB 9.1MB +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