-
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] Deprecate outdated detection rules Bulk APIs #129448
Conversation
b552997
to
24a7839
Compare
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
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.
LGTM, just a few comments @xcrzx
...s/security_solution/server/lib/detection_engine/routes/rules/create_rules_bulk_route.test.ts
Show resolved
Hide resolved
...s/security_solution/server/lib/detection_engine/routes/rules/create_rules_bulk_route.test.ts
Show resolved
Hide resolved
x-pack/test/detection_engine_api_integration/basic/tests/create_rules_bulk.ts
Show resolved
Hide resolved
x-pack/test/detection_engine_api_integration/security_and_spaces/tests/create_rules_bulk.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/server/lib/detection_engine/routes/rules/utils/deprecation.ts
Outdated
Show resolved
Hide resolved
24a7839
to
5cb5c6d
Compare
5cb5c6d
to
f356856
Compare
💚 Build SucceededTest Failures
Metrics [docs]Page load bundle
History
To update your PR or re-run it, just comment with: cc @xcrzx |
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.
doclink LGTM
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.
Awesome, thank you so much!
I just reviewed the latest changes, I didn't test the PR.
x-pack/plugins/security_solution/server/lib/detection_engine/routes/rules/utils/deprecation.ts
Show resolved
Hide resolved
x-pack/test/detection_engine_api_integration/security_and_spaces/tests/delete_rules_bulk.ts
Show resolved
Hide resolved
* @returns string | ||
*/ | ||
export const buildDeprecatedBulkEndpointMessage = (path: string) => { | ||
const docsLink = getDocLinks({ kibanaBranch: 'main' }).siem.ruleApiOverview; |
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.
wonder, is it might be better to give a link to a latest stable release, rather main branch?
(cherry picked from commit a897747)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
Addresses: #127756
Summary
The following endpoints are not used from Kibana anymore and have been deprecated:
detection_engine/rules/_bulk_create
detection_engine/rules/_bulk_delete
detection_engine/rules/_bulk_update
To let our users know about these deprecations we: