Add TCP support - Githubissues

talevy commented 9 years ago

Graylog 0.10 starts supporting TCP transport for gelf. Since this enables safer transport than UDP, I suggest extending the gelf input/output with a TCP option.

migrated from: https://logstash.jira.com/browse/LOGSTASH-780

maxhyjal commented 9 years ago

Is TCP option going to be supported anytime soon?

daniilyar commented 9 years ago

+1, this blocks us with propagation of Icinga2 perf. data to Logstash via GELF protocol. Icinga2 GELF plugin supports tcp only so I am wondering if TCP support is expected to be implemented soon.

maxhyjal commented 9 years ago

Will this be supported anytime soon? UDP isn't really reliable if I want to collect metrics via gelf input

maxhyjal commented 9 years ago

@jordansissel not sure if I should @ you here but who would be the best person to talk to? or probably this is of lower priority and then we can fix it and request a merge?

daniilyar commented 9 years ago

We've added TCP support recently. We will do some code cleanup soon and then make a PR. Until then here is a link to updated plugin code with quick 'howto' attached as comment: https://gist.github.com/daniilyar/758857675856ee2fdbef . Tested with Logstash 1.5.0 + Icinga 2.3.8

maxhyjal commented 9 years ago

Excellent @daniilyar !

fraenki commented 8 years ago

@daniilyar Ping. Care to submit a PR? People are already using your code... so this would make sense, I guess.

dracontb commented 8 years ago

Do I need to install icinga for this code to work? Isn't gelf plugin packaged with logstash 2.2 already supporting TCP? If so, what configuration to make it work?

shoutcool commented 7 years ago

would be great to see tcp support coming to gelf plugin soon.

jordansissel commented 7 years ago

I have deleted all of the "+1" comments. At the time of deletion, there were 12 +1 comments.

iFedix commented 7 years ago

Hey guys! Based on @daniilyar files, i adapted his file to last logstash version (5.4). It's working but it's a porting from an old version of the plugin, so lot of things changed and maybe it isn't optimized.. Also I did not test the udp part of the script, because i needed only tcp (i need nxlog tcp gelf to connect to logstash). You can download the file here: logstash-input-gelf-master.zip. Maybe it's a good start to add this in the current logstash-input-gelf plugin! Hope this helps someone! 😄

patricklucas commented 7 years ago

Any update on this?

iFedix commented 7 years ago

@patricklucas this is my working updated version based on the last offical release: logstash-input-gelf-master.zip. It's working well!

Hermain commented 6 years ago

The docker gelf loging driver now supports tcp output. Logstash needs to be able to receive that. https://docs.docker.com/engine/admin/logging/gelf/#gelf-options

omermizrahi commented 6 years ago

Hi does this work for somone? i try to work with gelf tcp input but get this in logs: [[main]<gelf] WARN logstash.inputs.gelf - Starting gelf listener (tcp) and no logs are processing when sent to TCP. im working in a logstash docker and installed the custom gelf-input.

jsvd commented 6 years ago

implemented in https://github.com/logstash-plugins/logstash-input-gelf/pull/59 and shipped already in version 3.1.0 to rubygems.org

logstash-plugins / logstash-input-gelf

Add TCP support #6