-
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
[Security Solution] Disable deprecated rules bulk CRUD API endpoints in Serverless and 9.0 #197422
[Security Solution] Disable deprecated rules bulk CRUD API endpoints in Serverless and 9.0 #197422
Conversation
2a35118
to
c6e7615
Compare
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
6979982
to
05b91bd
Compare
05b91bd
to
21f1e87
Compare
await Promise.all([ | ||
securitySolutionApi | ||
.createRule({ body: getCustomQueryRuleParams({ rule_id: 'rule-id-1' }) }) | ||
.expect(200), | ||
securitySolutionApi | ||
.createRule({ body: getCustomQueryRuleParams({ rule_id: 'rule-id-2' }) }) | ||
.expect(200), | ||
]); |
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.
I'm thinking if these changes to rules export tests should go in a separate PR so that we could backport them to 8.x
to decrease the chance of introducing conflicts with any other follow-up PRs related to rule import/export that will be merged to 8.x
.
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.
Ok, I'll open a new PR against 8.x
after merging this one 👍
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.
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.
Tested all deleted routes locally and received a 404
error for everything, lgtm!
💚 Build Succeeded
Metrics [docs]
History
cc @banderror |
…rt tests (#198953) **Follow-up to:** #197422 (comment) ## Summary This PR manually "backports" some of the changes from #197422 directly to the `8.x` branch. Specifically, it removes API calls to the rules `_bulk_create` API endpoint from the tests for exporting rules. This API endpoint has been disabled in `main` (=> Serverless and Kibana 9.0) and should not be used in code. This PR aims to reduce the chances of introducing conflicts with any further changes to rule export tests.
Partially addresses: #193184
Breaking change proposal: https://github.com/elastic/dev/issues/2772 (internal)
Summary
This PR disables the following deprecated bulk API endpoints for creating, updating and deleting detection rules from Elastic Security APIs in Serverless and upcoming
v9.0.0
:Specifically, as a first step we remove the endpoints from the route registrations. Once https://github.com/elastic/dev/issues/2772 is approved, we will merge this PR and remove the corresponding endpoint handlers and associated code in a follow-up PR.
Checklist
For maintainers