pinojs / pino-elasticsearch

🌲 load pino logs into Elasticsearch
MIT License
179 stars 67 forks source link

Use the new Elasticsearch client #26

Closed delvedor closed 5 years ago

delvedor commented 5 years ago

As titled.

This pr also removes the aws configuration, since the package has not been updated to support the new client. It introduces two new cli options:

It removes host, port, aws-credentials.

The minimum version of Node.js required is v8.

mcollina commented 5 years ago

Failing on Node 12: https://travis-ci.org/pinojs/pino-elasticsearch/jobs/541274020

delvedor commented 5 years ago

I saw that, but the test is passing on my machine. I fear is a timing issue. Can you rerun the CI job?

delvedor commented 5 years ago

Now CI is green. Unfortunately, Travis may cause this flaky tests from time to time.