-
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
[Entity Analytics] Asset criticality bulk upload JSON API #186310
Conversation
/ci |
35f51d5
to
194b01b
Compare
Pinging @elastic/security-entity-analytics (Team:Entity Analytics) |
...st/security_solution_api_integration/test_suites/entity_analytics/utils/asset_criticality.ts
Outdated
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.
Just a small question
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!
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @hop-dev |
Summary
Add the asset criticality bulk upload API at
POST /api/asset_criticality/bulk
. Records supplied will either be created or updated.This API has the same functionality as the csv upload API but takes a JSON body.
A couple slight differences:
Example curl (note asset criticality advanced setting must be enabled otherwise you will get a 403):
I have added API tests for the new route