Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bug in EventTransformCommon when using the multiline codec #73

Closed
guyboertje opened this issue Apr 11, 2016 · 2 comments
Closed

Bug in EventTransformCommon when using the multiline codec #73

guyboertje opened this issue Apr 11, 2016 · 2 comments
Assignees
@guyboertje

Comments

@guyboertje
Copy link

Exception:

Multiline: flush downstream error {:exception=>#<ArgumentError: wrong number of arguments calling `config` (0 for 1)>

From logstash issues 5084...

beats-input handle_new_connection ensure block calls flush with a block
multiline flush yields events to the block given.
block callback calls transformer.transform
DecodedEventTransform#transform calls codec_name
EventTransformCommon#codec_name calls @input.codec.base_codec.class.config

Should be @input.codec.base_codec.class.config_name

https://github.com/logstash-plugins/logstash-input-beats/blob/master/lib/logstash/inputs/beats_support/event_transform_common.rb#L36
@guyboertje guyboertje mentioned this issue Apr 11, 2016
Closed
@ph ph self-assigned this Apr 11, 2016
@ph
Copy link
Contributor

ph commented Apr 11, 2016

/facepalm

@ph ph assigned guyboertje and unassigned ph Apr 11, 2016
@ph ph changed the title Bug in EventTransformCommon#codec_name causes Multiline: flush downstream error {:exception=>#<ArgumentError: wrong number of arguments calling config (0 for 1)> Bug in EventTransformCommon when using the multiline codec Apr 11, 2016
guyboertje pushed a commit to guyboertje/logstash-input-beats that referenced this issue Apr 12, 2016
Fix for logstash-plugins#73
f5f08cc
guyboertje pushed a commit that referenced this issue Apr 12, 2016
Merge pull request #75 from guyboertje/fix/73
b2a867f 
Fix for #73
@guyboertje
Copy link
Author

Closed by f5f08cc

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

@guyboertje guyboertje

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ph @guyboertje