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.

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

netflow codec should override the encode method from the base filter class #196

Open
TheVastyDeep opened this issue Dec 1, 2021 · 1 comment

Comments

@TheVastyDeep
Copy link

logstash 7.15.2, installed using yum, run from the command line. The following configuration

input { generator { count => 1 lines => [ '' ] } }
output { udp { port  => 2055 codec => netflow host => "127.1.2.3" } }

results in a stack overflow, as noted in this issue raised against logstash-core, because of recursion between encode and multi-encode is the base filter.

It would be trivial to fix this in the codec by overriding encode with a method that raises an exception. The dots codec shows how to do this.

@kares
Copy link
Contributor

kares commented Dec 2, 2021

thanks, for reference it's also been feature requested to support encoding with the codec #185

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

No branches or pull requests

2 participants