-
Notifications
You must be signed in to change notification settings - Fork 460
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
[Carbon Black Cloud] Cleanup fields based on Alert API doc #5927
[Carbon Black Cloud] Cleanup fields based on Alert API doc #5927
Conversation
964b952
to
7e8023f
Compare
🌐 Coverage report
|
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.
Can you note in the commit the provenance of the new test samples. I'm assuming that they are just the old ones with the defunct fields removed.
Hey @efd6 I dont have the new test samples.
Yes |
Removes fields missing from the Carbon Black Cloud Alert APIs(https://developer.carbonblack.com/reference/carbon-black-cloud/platform/latest/alerts-api/#fields.) but present in the integration doc
7e8023f
to
df0655c
Compare
Package carbon_black_cloud - 1.7.1 containing this change is available at https://epr.elastic.co/search?package=carbon_black_cloud |
Removes fields missing from the Carbon Black Cloud Alert APIs(https://developer.carbonblack.com/reference/carbon-black-cloud/platform/latest/alerts-api/#fields.) but present in the integration doc
What does this PR do?
event.url
as there is noalert_url
field anymore in the API doc.carbon_black_cloud.alert.device.external_ip
as there is nodevice_external_ip
field anymore in the API doc.carbon_black_cloud.alert.device.internal_ip
as there is nodevice_internal_ip
field anymore in the API doc.Checklist
changelog.yml
file.How to test this PR locally
elastic-package stack down && elastic-package build && elastic-package stack up --version=8.8.0-SNAPSHOT -d -v && eval "$(elastic-package stack shellinit)" && elastic-package test -v
Related issues
Screenshots