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

Please rollout the new patch version for this plugin to the upstream #71

Open
darajnish opened this issue Mar 5, 2024 · 0 comments
Open
Labels

Comments

@darajnish
Copy link

The current upstream version 3.0.5 has the following code in the register function:

if @codec.instance_of? LogStash::Codecs::Plain
end

which always returns false and so the @format of the LogStash::Codecs::Plain codec is never defined, making the message config option of the plugin useless as it's always with the default value %{message}.

Further, since there is no @format defined for LogStash::Codecs::Plain, the encode function of the codec always relies on event.to_s which results in syslog actual payload becoming %{timestamp} %{host} %{message}. And, this is always prepending the %{timestamp} %{host} part which breaks the format of the desired syslog payload.

I'm using this plugin to send high volume logs to our other critical systems that use the messages received via syslog to identify various kinds of patterns in the logs. It would make our work easier and would benefit everyone if the current changes are pushed to the upstream with a new patch version. Thanks

@darajnish darajnish added the bug label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant