[Security Solution] Fixes legacy rules/prepackaged endpoints from returning 500 on Basic license when dealing with ML rules#253574
Merged
dplumlee merged 5 commits intoelastic:mainfrom Feb 20, 2026
Conversation
dplumlee
added
bug
Contributor
|
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Contributor
|
Pinging @elastic/security-solution (Team: SecuritySolution) |
Contributor
|
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
Member
|
Hi Davis, thanks for addressing this oustanding license restriction issue with legacy prebuilt rule installation URLs. The code changes look great, you have addressed existing I have tested both endpoints before and after your changes and I can confirm that the 
|
sdesalas
approved these changes
Feb 20, 2026
Contributor
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
cc @dplumlee |
Contributor
|
Starting backport for target branches: 8.19, 9.2, 9.3 https://github.com/elastic/kibana/actions/runs/22238363758 |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Feb 20, 2026
…eturning 500 on Basic license when dealing with ML rules (elastic#253574) ## Summary Addresses elastic#253340 Adds filters to the `rules/prepackaged` and `rules/prepackaged/_status` API endpoints to guard against use cases where users are on basic licenses and are trying to install or update rules that include ML rules. Currently ML rules are limited to platinum licenses and should not be included in these endpoint responses. The inclusion of these rules in all cases were causing 500 errors every time the calls were made, even if all license-available rules were installed successfully. This PR also adds unit and FTR tests to cover the fixed legacy endpoint logic ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: Tiago Costa <tiago.costa@elastic.co> (cherry picked from commit ea1da14)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Feb 20, 2026
…eturning 500 on Basic license when dealing with ML rules (elastic#253574) ## Summary Addresses elastic#253340 Adds filters to the `rules/prepackaged` and `rules/prepackaged/_status` API endpoints to guard against use cases where users are on basic licenses and are trying to install or update rules that include ML rules. Currently ML rules are limited to platinum licenses and should not be included in these endpoint responses. The inclusion of these rules in all cases were causing 500 errors every time the calls were made, even if all license-available rules were installed successfully. This PR also adds unit and FTR tests to cover the fixed legacy endpoint logic ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: Tiago Costa <tiago.costa@elastic.co> (cherry picked from commit ea1da14)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Feb 20, 2026
…eturning 500 on Basic license when dealing with ML rules (elastic#253574) ## Summary Addresses elastic#253340 Adds filters to the `rules/prepackaged` and `rules/prepackaged/_status` API endpoints to guard against use cases where users are on basic licenses and are trying to install or update rules that include ML rules. Currently ML rules are limited to platinum licenses and should not be included in these endpoint responses. The inclusion of these rules in all cases were causing 500 errors every time the calls were made, even if all license-available rules were installed successfully. This PR also adds unit and FTR tests to cover the fixed legacy endpoint logic ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: Tiago Costa <tiago.costa@elastic.co> (cherry picked from commit ea1da14)
Contributor
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
kibanamachine
added a commit
that referenced
this pull request
Feb 20, 2026
… from returning 500 on Basic license when dealing with ML rules (#253574) (#254291) # Backport This will backport the following commits from `main` to `8.19`: - [[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)](#253574) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Davis Plumlee","email":"56367316+dplumlee@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-02-20T19:37:22Z","message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:fix","Team:Detections and Resp","Team: SecuritySolution","Team:Detection Rule Management","Feature:Prebuilt Detection Rules","backport:version","v9.4.0","v8.19.13","v9.2.7","v9.3.2"],"title":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules","number":253574,"url":"https://github.com/elastic/kibana/pull/253574","mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},"sourceBranch":"main","suggestedTargetBranches":["8.19","9.2","9.3"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/253574","number":253574,"mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},{"branch":"8.19","label":"v8.19.13","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.7","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.3","label":"v9.3.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Davis Plumlee <56367316+dplumlee@users.noreply.github.com> Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Feb 20, 2026
…from returning 500 on Basic license when dealing with ML rules (#253574) (#254292) # Backport This will backport the following commits from `main` to `9.2`: - [[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)](#253574) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Davis Plumlee","email":"56367316+dplumlee@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-02-20T19:37:22Z","message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:fix","Team:Detections and Resp","Team: SecuritySolution","Team:Detection Rule Management","Feature:Prebuilt Detection Rules","backport:version","v9.4.0","v8.19.13","v9.2.7","v9.3.2"],"title":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules","number":253574,"url":"https://github.com/elastic/kibana/pull/253574","mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},"sourceBranch":"main","suggestedTargetBranches":["8.19","9.2","9.3"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/253574","number":253574,"mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},{"branch":"8.19","label":"v8.19.13","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.7","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.3","label":"v9.3.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Davis Plumlee <56367316+dplumlee@users.noreply.github.com> Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Feb 20, 2026
…from returning 500 on Basic license when dealing with ML rules (#253574) (#254293) # Backport This will backport the following commits from `main` to `9.3`: - [[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)](#253574) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Davis Plumlee","email":"56367316+dplumlee@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-02-20T19:37:22Z","message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:fix","Team:Detections and Resp","Team: SecuritySolution","Team:Detection Rule Management","Feature:Prebuilt Detection Rules","backport:version","v9.4.0","v8.19.13","v9.2.7","v9.3.2"],"title":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules","number":253574,"url":"https://github.com/elastic/kibana/pull/253574","mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},"sourceBranch":"main","suggestedTargetBranches":["8.19","9.2","9.3"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/253574","number":253574,"mergeCommit":{"message":"[Security Solution] Fixes legacy `rules/prepackaged` endpoints from returning 500 on Basic license when dealing with ML rules (#253574)\n\n## Summary\n\nAddresses https://github.com/elastic/kibana/issues/253340\n\nAdds filters to the `rules/prepackaged` and `rules/prepackaged/_status`\nAPI endpoints to guard against use cases where users are on basic\nlicenses and are trying to install or update rules that include ML\nrules.\n\nCurrently ML rules are limited to platinum licenses and should not be\nincluded in these endpoint responses. The inclusion of these rules in\nall cases were causing 500 errors every time the calls were made, even\nif all license-available rules were installed successfully.\n\nThis PR also adds unit and FTR tests to cover the fixed legacy endpoint\nlogic\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n\n---------\n\nCo-authored-by: Tiago Costa <tiago.costa@elastic.co>","sha":"ea1da14507a291fc6f8cf557c7ba18e5c8ef42cb"}},{"branch":"8.19","label":"v8.19.13","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.2","label":"v9.2.7","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.3","label":"v9.3.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Davis Plumlee <56367316+dplumlee@users.noreply.github.com> Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
Contributor
|
@dplumlee Please review the version labels and fix, if needed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses #253340
Adds filters to the
rules/prepackagedandrules/prepackaged/_statusAPI endpoints to guard against use cases where users are on basic licenses and are trying to install or update rules that include ML rules.Currently ML rules are limited to platinum licenses and should not be included in these endpoint responses. The inclusion of these rules in all cases were causing 500 errors every time the calls were made, even if all license-available rules were installed successfully.
This PR also adds unit and FTR tests to cover the fixed legacy endpoint logic
Checklist
Check the PR satisfies following conditions.
Reviewers should verify this PR satisfies this list as well.