Skip to content

Add support for TLS settings - #35

Closed
slauger wants to merge 7 commits into
logstash-plugins:masterfrom
slauger:master
Closed

Add support for TLS settings#35
slauger wants to merge 7 commits into
logstash-plugins:masterfrom
slauger:master

Conversation

@slauger

@slauger slauger commented Oct 23, 2018

Copy link
Copy Markdown

The rubygem gelf library supports ssl since it's version 3.0.0. This PR adds new SSL options to logstash-output-gelf. This is pretty much a reimplementation of #28.

Parameters

  • ssl (boolean)
  • ssl_certificate_authorities (string)
  • ssl_certificate (string)
  • ssl_key (string)
  • ssl_verify_mode ([none, peer, force_peer])

Currently there is no option for ssl_ciphers as the gelf library dosen't have any kind of cipherlist option.

Please review and let me know if any changes are necessary.

@slauger

slauger commented Nov 6, 2018

Copy link
Copy Markdown
Author

Hi, i just signed the CLA. Let me know if any further changes are necessary.

@slauger

slauger commented Nov 17, 2018

Copy link
Copy Markdown
Author

Any update or feedback? Is there anything i can do to increase the chance of merging?

@KurzweilTV

Copy link
Copy Markdown

Is anyone available to look at this request? Having this support for the gelf output would be highly valuable.

@slauger

slauger commented Jan 17, 2019

Copy link
Copy Markdown
Author

I'm going to close this because i'm no longer working on this. Because of the poor support we decided to move away from logstash-output-gelf. We also had some performance issues with the GELF output. If anyone is interested feel free to "fork my fork".

The main reason for this was to get our stuff from Logstash into our Graylog instance. We finally resolved this by putting a RabbitMQ instance between Logstash and Graylog (and using logstash-output-rabbitmq).

@slauger slauger closed this Jan 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants