search

pinojs / pino-elasticsearch

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

Upgrade to ElasticSearch 8 #146

Closed mcollina closed 2 years ago

mcollina commented 2 years ago

ElasticSearch 8 now requires https for every communication. This makes the current approach we use for running tests moot.

We need some new setup and I do not have time to look into it right now.

@delvedor can you help?

delvedor commented 2 years ago

I've updated elastic/elastic-github-actions to better support security on by default. While testing the code should either avoid rejecting unauthorized certs, or set security-enabled: false in the action settings.

mcollina commented 2 years ago

@Fdawgs could you take a look at this? https://github.com/fastify/fastify-elasticsearch was recently updated.

Fdawgs commented 2 years ago

@Fdawgs could you take a look at this? https://github.com/fastify/fastify-elasticsearch was recently updated.

Would it just be the CI that needs an update? Not familiar with Elasticsearch at all.

mcollina commented 2 years ago

They switched from being http to being https by default in the latest version.