Closed olivierlemasle closed 1 year ago
/assign @dgrisonnet
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dgrisonnet, olivierlemasle
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Having no explicit
MinVersion
is reported by gosec as G402 (CWE-295):TLS MinVersion too low
Using MinVersion: tls.VersionTLS12 because it's what client-go uses:
https://github.com/kubernetes/client-go/blob/1ac8d459351e21458fd1041f41e43403eadcbdba/transport/transport.go#L92
That way, the Kubernetes API client and the Prometheus client in
prometheus-adapter
use the same TLS configMinVersion
.