[SIEM][Detection Engine][Lists] Adds specific endpoint_list REST API and API for abilities to auto-create the endpoint_list if it gets deleted#71792
Merged
peluja1012 merged 9 commits intoelastic:masterfrom Jul 15, 2020
Conversation
…ate the endpoint_list if it gets deleted
2 tasks
FrankHassanabad
commented
Jul 15, 2020
FrankHassanabad
commented
Jul 15, 2020
x-pack/plugins/lists/common/schemas/request/update_endpoint_list_item_schema.ts
Outdated
Show resolved
Hide resolved
FrankHassanabad
commented
Jul 15, 2020
x-pack/plugins/lists/server/services/exception_lists/create_endpoint_list.ts
Outdated
Show resolved
Hide resolved
spong
reviewed
Jul 15, 2020
Comment on lines
+58
to
+61
|
|
||
| // This will create the endpoint list if it does not exist yet | ||
| await context.lists?.getExceptionListClient().createEndpointList(); | ||
|
|
Contributor
|
Pinging @elastic/siem (Team:SIEM) |
rylnd
approved these changes
Jul 15, 2020
Contributor
rylnd
left a comment
There was a problem hiding this comment.
Tested the "create endpoint list during rule creation" flow, along with the "associate endpoint list to rule" flow, and both look great! 👍 👍 👍
Up to you if you wanna merge this now and add tests afterwards. Either way I'm gonna have to circle back on #71794, mainly to use the ENDPOINT_LIST_ID constant.
Member
|
@elasticmachine merge upstream |
7 tasks
Contributor
|
@elasticmachine merge upstream |
Contributor
💚 Build SucceededBuild metrics@kbn/optimizer bundle module count
miscellaneous assets size
page load bundle size
History
To update your PR or re-run it, just comment with: |
gmmorris
added a commit
to gmmorris/kibana
that referenced
this pull request
Jul 15, 2020
* master: (82 commits) Fixed the spacing of child accordion items for policy response dialog. (elastic#71677) [SECURITY] Timeline bug 7.9 (elastic#71748) use fixed isChromeVisible method (elastic#71813) [SIEM][Detection Engine][Lists] Adds specific endpoint_list REST API and API for abilities to auto-create the endpoint_list if it gets deleted (elastic#71792) [test] Skips flaky Saved Objects Management test [APM] Remove watcher integration (elastic#71655) [APM] Increase `xpack.apm.ui.transactionGroupBucketSize` (elastic#71661) [test] Skips Ingest Manager test preventing ES promotion [test] Skips flaky detection engine tests Revert "re-fix navigate path for master add SAML login to login_page (elastic#71337)" [tests] Temporarily skipped Fleet tests [test] Skipped monitoring test [Security Solution][Detections] Associate Endpoint Exceptions List to Rule during rule creation/update (elastic#71794) Add endpoint exception creation API validation (elastic#71791) Skip jest tests that timeout waiting for react (elastic#71801) [Security Solution][Exceptions] - Adds filtering to endpoint index patterns by exceptional fields (elastic#71757) [Reporting] Re-delete a file (elastic#71730) [Security Solution] [Detections] Fixes bug for determining when we hit max signals after filtering with lists (elastic#71768) [Ingest Manager] Better display of Fleet requirements (elastic#71686) [tests] Temporarily skipped to promote snapshot ...
Contributor
|
Pinging @elastic/security-solution (Team: SecuritySolution) |
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
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
Adds specific endpoint list REST API and API for abilities to auto-create the endpoint_list if it gets deleted or to auto-provision it when a rule is first created.
When auto-creating the endpoint_list it does so using these values for the saved object:
The new API endpoints are as follows below. These endpoints are like the normal exception list item endpoints except you should not pass down any
list_idornamespace_typesince these are focused specific endpoints against the space agnostic endpoint_listNew API within the exception_list_client plugin are:
Scripts for testing are:
Checklist
Delete any items that are not applicable to this PR.