-
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
New Connector Type - D3 security SOAR #144955
New Connector Type - D3 security SOAR #144955
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
x-pack/plugins/stack_connectors/server/connector_types/stack/d3security/index.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/stack_connectors/server/connector_types/stack/d3security/index.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.
initial build failures:
console.log
not allowed, please remove or use suitable log mocking
@elastic/response-ops could you please take a look at the proposal? |
Hi @TinaHeiligers, we have already removed 'console.log' in the latest commits. Please, review it again. Thank you! |
@elasticmachine merge upstream |
@elastic/response-ops could you please take a look at the proposal? |
@TinaHeiligers Hi Tina, any update from the code review? Is there anything we can help with? |
@d3-naiji I am removing myself as a reviewer to allow the relevant team to take this further. |
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.
CC @elastic/response-ops
Hey @d3-bensong! Could you please address the CLA issue? Thanks! |
Hi @cnasikas , I helped him sign the CLA, but it's after his commits. Could you help check if it can re-run the CLA checker and do I need to sign CLA for the developer again? |
@elasticmachine merge upstream |
buildkite test this |
@elasticmachine, run elasticsearch-ci/docs |
@d3-naiji It seems that the CLA is fixed now. For some weird reason, I cannot merge with upstream. Could you please do it manually? |
@d3-naiji @d3-bensong Is there any plan to integrate the connector with Cases? |
Hi @cnasikas , |
buildkite test this |
@elasticmachine, run elasticsearch-ci/docs |
Hi @cnasikas , |
buildkite test this |
@elasticmachine, run elasticsearch-ci/docs |
💔 Build FailedFailed CI StepsMetrics [docs]Module Count
Async chunks
Page load bundle
Unknown metric groupsasync chunk count
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
Hi @cnasikas, |
Hello @d3-bensong and @d3-naiji. After review, because of the planned cases enhancements it was determined that the D3 Security connector needed to be transitioned to our sub actions framework. In order to move this PR forward in time for our upcoming 8.9 Feature Freeze date, we determined it would be most efficient if we move forward with an Elastic owned branch and have our team carry out the rest of the work. There is a new PR open with the sub actions refactor complete, along with additional required tests and documentation. We are moving forward with a review on that PR today. Your work is included in that PR. Therefore, you can go ahead and close this one. I will follow up over email to confirm once I have the new PR merged. Let me know if you have any questions. Thank you! |
@stephmilovic Thanks! I will close this PR |
Summary
Introduces a new built-in action type and sends the ElasticSearch alerts to D3 Security SOAR for incident response, workflow automation and deep investigation. This integrated alert action allows you to create Events or trigger Playbook workflow actions automatically in the D3 Security SOAR platform.
The new action type and the action connector with D3 Security SOAR are as follows:
and action params form available in Create Alert form:
OBS.: merge conflict has been resolved and new fields have been added for params form.
Checklist
Delete any items that are not applicable to this PR.
For maintainers
(https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)