Cherry-pick #18392 to 7.8: Cisco asa/ftd: Remove _temp_ fields on failure #18490
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.
Cherry-pick of PR #18392 to 7.8 branch. Original message:
What does this PR do?
Updates the shared cisco asa/ftd ingest pipeline to remove the fields under
_temp_
in the case of failure.Why is it important?
Not removing the temporary fields can cause the index mapping to grow too big.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
Unfortunately I couldn't add a test because Filebeat module's tests don't allow documents that cause errors.
Tested it locally by adding a test file that would cause an error in the pipeline:
And confirmed that with the fix,
_temp_
fields are no longer part of the output document:while before this patch one would get some of them:
Related issues