logstash-plugins / logstash-filter-elasticsearch

Elasticsearch filter for Logstash
Apache License 2.0
21 stars 83 forks source link

Fix: hosts default + client gem 7.x compatibility #148

Closed kares closed 3 years ago

kares commented 3 years ago

These changes primarily fix the regression in LS 7.14.0 where due the elasticsearch client update (from 5.0.5 to 7.5.0) the Authorization header isn't passed (due a bug in client versions 7.2.0 - 7.14.0). This leads to the plugin not being able to leverage user/password credentials set by the user to connect to ES.

The fix involves a patch to work-around the elasticsearch client issue. Docker was adjusted to be able to run ES secured with $ELASTIC_PASSWORD for the elastic user (for integration tests).


We're also fixing a small issue that I've run into where omitting the hosts option wasn't working properly (with the localhost:9200 default) and another plugin bug due @hosts variable mutation (from multiple workers).

resolves #147 resolves #125 resolves #129 (closes https://github.com/logstash-plugins/logstash-filter-elasticsearch/pull/133)