-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SIEM] Add license checks for ML Rules on the backend #61023
Conversation
We'll add this to the rest of the routes momentarily.
This ensures that you cannot create nor update an ML Rule if your license is not Platinum (or Trial).
Pinging @elastic/siem (Team:SIEM) |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/context/_discover_navigation·js.context app context link in discover should open the context view with the selected document as anchorStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
x-pack/legacy/plugins/siem/server/lib/detection_engine/routes/rules/create_rules_bulk_route.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked out, tested locally, and performed code review. Thanks for the clean documented code and for adding tests to each area impacted -- greatly appreciated @rylnd! LGTM 👍
* WIP: Check license on simple rule creation We'll add this to the rest of the routes momentarily. * Add license checks around all rule-modifying endpoints This ensures that you cannot create nor update an ML Rule if your license is not Platinum (or Trial). Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Pinging @elastic/security-solution (Team: SecuritySolution) |
Summary
Rejects creation/update/importing of ML Rules if the license is insufficient.
Checklist
Delete any items that are not applicable to this PR.
For maintainers