markmcdowell / NLog.Targets.ElasticSearch

NLog target for Elasticsearch
MIT License
176 stars 89 forks source link

Adds support for elastic cloud ids. #108

Closed christophano closed 5 years ago

christophano commented 5 years ago

Adds support for cloud id connection pool. Things to note: username and password authentication is required when using cloud id connection pool but I haven't checked that they're provided - the connection should just fail authentication.

I also don't check the RequireAuth property, as we know it is implicitly true in this scenario. If RequireAuth is also set to true then the username and password will be applied again (with the same values). Not sure whether this would cause any problems downstream, but I suspect not.