statsbomb/prometheus-msk-discovery

invalid memory address or nil pointer dereference when MSK only accepts TLS

Closed this issue · 3 comments

Apparently BootstrapBrokerString doesn't exist when the MSK cluster only accepts TLS.
By the SDK docs it should be in BootstrapBrokerStringTls but I don't know much Go to offer a pull request on this

Thanks for the report @Thiago-Dantas. This should be fixed now.

You should probably add kafka:ListNodes as a required IAM permission on the README

Ahh good catch, thanks!