Skip to content

[WIP] Encoder drop additional fields#57

Open
IngaFeick wants to merge 26 commits intologstash-plugins:mainfrom
IngaFeick:encoder_drop_additional_fields
Open

[WIP] Encoder drop additional fields#57
IngaFeick wants to merge 26 commits intologstash-plugins:mainfrom
IngaFeick:encoder_drop_additional_fields

Conversation

@IngaFeick
Copy link
Copy Markdown
Collaborator

@IngaFeick IngaFeick commented Jun 19, 2020

This PR introduces a new config option to automatically discard event fields which are not in the protobuf definition. In the current version 1.2.4 an event with additional fields will be discarded and an error will be logged. This PR adds the option pb3_encoder_drop_unknown_fields. When activated (default) it will remove incorrect fields from the event before attempting the encoding.

@IngaFeick IngaFeick changed the title Encoder drop additional fields [WIP] Encoder drop additional fields Jul 10, 2020
@IngaFeick
Copy link
Copy Markdown
Collaborator Author

I'm setting this to WIP because we have discovered an NPE when using this in production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant