search

Telefonica / prometheus-kafka-adapter

Use Kafka as a remote storage database for Prometheus (remote write only)
Apache License 2.0
364 stars 135 forks source link

Support TLS in addition to Mutual TLS #118

Open rmadamson opened 1 year ago

rmadamson commented 1 year ago

In main.go, on line 38, all three configuration options must be specified in order to set the appropriate kafka ca certificate location parameter. Mutual TLS requires all three, but regular TLS (where the brokers don't authenticate the clients over TLS) does not work unless you create a bogus client certificate and keypair (that isn't used).

An easy fix would be to set those TLS options if the kafka security protocol is either ssl or sasl_ssl

palmerabollo commented 1 year ago

@rmadamson do you feel like contributing this change to the repo?