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

Typo in logstash/elasticsearch TLS configuration #23

Closed
stith opened this issue Apr 14, 2016 · 3 comments
Closed

Typo in logstash/elasticsearch TLS configuration #23

stith opened this issue Apr 14, 2016 · 3 comments
Labels

Comments

@stith
Copy link

stith commented Apr 14, 2016

Howdy y'all!

Found a quick typo in the filebeat.yaml templates for logstash and elasticsearch's TLS configuration:

https://github.com/pcfens/puppet-filebeat/blob/master/templates/filebeat.yml.ruby18.erb#L81-L83
https://github.com/pcfens/puppet-filebeat/blob/master/templates/filebeat.yml.ruby18.erb#L137-L139

Looks like you're checking for certifcate_key (typo, missing i in certificate) and then using certificate_key (spelled correctly) in the actual output. 😄

@stith stith changed the title Typo in logstash TLS configuration Typo in logstash/elasticsearch TLS configuration Apr 14, 2016
@pcfens pcfens added the bug label Apr 14, 2016
@pcfens pcfens closed this as completed in 316d228 Apr 14, 2016
@pcfens
Copy link
Owner

pcfens commented Apr 14, 2016

Thanks for finding this - I've patched and pushed a new release up to the forge as v0.6.1

@stith
Copy link
Author

stith commented Apr 14, 2016

Thanks for the quick fix! You missed the logstash one though, which is the one that I'm actually trying to use hehehe.

https://github.com/pcfens/puppet-filebeat/blob/master/templates/filebeat.yml.ruby18.erb#L137-L139

@pcfens
Copy link
Owner

pcfens commented Apr 15, 2016

Oops :) - moving a little too fast.

I've pushed this fix up as v0.6.2.

Thanks again.

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

2 participants