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

Fluentd / FluentBit out_forward compat #9

Merged
merged 2 commits into from
Jan 27, 2017

Conversation

MerlinDMC
Copy link
Contributor

This does add compatibility with later Fluentd and FluentBit out_forward encodings.
Both out_forward streams are sending combined packets that bundle up multiple events for a single tag.

This currently does not add support for compressed streams. It will raise a failure and log it in the event stream.

This should also fix #4 and #2

@krallistic
Copy link

Any chance that this will get merged?

@joeris-CGM
Copy link

joeris-CGM commented Jan 27, 2017

+1, merge please

@jsvd jsvd merged commit 6734f1c into logstash-plugins:master Jan 27, 2017
@agonzalezro
Copy link

Could this be released as a gem? I bit the same error and the only solution I found was to use copy the file in master to the logstash container.

It seems that logstash is always using the latest version (I couldn't find a Gemfile.lock): https://github.com/elastic/logstash/blob/master/Gemfile#L36

In case you couldn't release, what would be the way to move this from my test environment to production? Override the gem in logstash with the master of this repo?

Thanks!

@ThoTischner
Copy link

+1 Please release this fix.

@jordansissel
Copy link
Contributor

I am attempting to do a release now.

@jordansissel
Copy link
Contributor

Version 3.1.1 of this plugin is now published.

@agonzalezro
Copy link

agonzalezro commented Mar 15, 2017

Awesome! Thanks @jordansissel

@hselvara
Copy link

Hi All,

I have a made a pull request for the avoiding overwriting of tags with key named as 'tags' with #10 .Any chance this will get merged?

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.

Codec kills listener on exception
8 participants