search

sensu / sensu-influxdb-handler

Sensu Go InfluxDB Metrics Handler
https://sensu.io
MIT License
14 stars 16 forks source link

Unauthenticated connections are not supported #23

Closed cwjohnston closed 5 years ago

cwjohnston commented 5 years ago

Expected behavior

handler can relay metrics to an InfluxDB instance without requiring changes to InfluxDB configuration

Current behavior

handler requires username and password to be provided, which in turn requires InfluxDB to be reconfigured with authentication enabled

Context

I am attempting to use this handler with InfluxDB out of the box -- i.e. without enabling authentication nor creating credentials (username, password).

Per the InfluxDB documentation, authentication is disabled by default.

I've found that the username and password are required settings, and haven't been successful in passing empty values for these settings.

nikkictl commented 5 years ago

Looks like the Influx client code states username and password are optional too. We should adhere to their default behavior in the absence of those flags.