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
[RecordedFuture] Add dashboards and list field #9471
Conversation
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
@@ -237,6 +237,10 @@ processors: | |||
- rename: | |||
field: json | |||
target_field: recordedfuture | |||
- set: |
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.
Should we have a test case that exercises this? Also, is it worth using a rename instead of a set?
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.
The field list
is added to both the system test configs already that has httpjson as input. I think the CSV config (file input) is running last which is being taken into sample_event.json
hence the field is missing in that file.
I changed the processor to rename
.
💚 Build Succeeded
History
cc @kcreddy |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Package ti_recordedfuture - 1.23.0 containing this change is available at https://epr.elastic.co/search?package=ti_recordedfuture |
Proposed commit message
Checklist
changelog.yml
file.Screenshots