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

Update gelfd dependecy to allow uncompressed input #48

Merged
merged 1 commit into from
Aug 14, 2019

Conversation

ptqa
Copy link
Contributor

@ptqa ptqa commented Sep 15, 2016

Hi.

Since gelfd maintainer is not releasing new version of gem for about 2 years (lusis/gelfd#4) I've made a fork and published it to rubygems as gelfd2 (https://github.com/ptqa/gelfd2). I will try to maintain it as long as I can, although there is not a lot of activity.

I've tested this gem on my setup and it seems to work.

@ptqa ptqa changed the title Update gelfd dependecy to allow uncopressed input Update gelfd dependecy to allow uncompressed input Sep 16, 2016
@bobzoller
Copy link

What's the official word here? I've now got my own fork of logstash-input-gelf for this reason, and I can see others doing the same.

@jordansissel
Copy link
Contributor

I'm open to merging this, but:

  • it currently has a merge conflict
  • it needs to be reviewed

@rikwasmus
Copy link
Contributor

#11 would be fixed with this.

@ptqa
Copy link
Contributor Author

ptqa commented Oct 17, 2017

@jordansissel I've resolved merge conflict and fixed docs. Is it good for merge now?

@jamtur01
Copy link

It'd be awesome to see this merged - the Gelf input is largely broken without it. :(

@pkoretic
Copy link

Any news on this one?
While I'm asking should this plugin be taken by some other maintainer?

@os11k
Copy link

os11k commented Aug 13, 2019

Any updates? I would be so happy if this can be merged.

@ptqa
Copy link
Contributor Author

ptqa commented Aug 14, 2019

@colinsurprenant @yaauie @robbavey Guys, as we approach 3rd anniversary for this PR, I have rebased it on current master & it's passing all CI tests now. I'm not changing CHANGELOG.md to avoid merge conflicts. Maybe we can merge this PR this year?:)

@jsvd jsvd merged commit 27ae51d into logstash-plugins:master Aug 14, 2019
@jsvd
Copy link
Member

jsvd commented Aug 14, 2019

thanks for the patience and the work you've done here, version 3.3.0 has been published with these changes https://rubygems.org/gems/logstash-input-gelf/versions/3.3.0

@kolyshkin
Copy link

This made its way to logstash 7.4, released Oct 1, 2019
https://www.elastic.co/guide/en/logstash/7.4/logstash-7-4-0.html

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.

10 participants