Better indicate deprecation of username/password authentication

Dabz / ccloudexporter

Prometheus exporter for Confluent Cloud API metric

https://docs.confluent.io/current/cloud/metrics-api.html

86 stars 53 forks source link

Better indicate deprecation of username/password authentication #39

Closed sirianni closed 4 years ago

sirianni commented 4 years ago

Username/password authentication to the Metrics API was only supported in the preview phase. For GA release, only API key/secret authentication is officially supported.

Rename CCLOUD_USER and CCLOUD_PASSSWORD environment variables to CCLOUD_APIKEY and CCLOUD_APISECRET respectively
- The legacy environment variables may still need to be supported to maintain backwards compatibility
Update the README accordingly

Dabz commented 4 years ago

I will do the change & the documentation (with a big disclaimer in the README) ASAP (normally tomorrow ;) ). I assume that username/password authentication will no longer be supported at some point, right?

sirianni commented 4 years ago

I assume that username/password authentication will no longer be supported at some point, right?

Correct. We are looking to sunset this later in Q3.