Skip to content

Commit a73f810

Browse files
committed
only register grpc TLS metrics on success
Signed-off-by: mattb18 <revoked>
1 parent 7e25c6f commit a73f810

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

prober/grpc.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,6 @@ func ProbeGRPC(ctx context.Context, target string, module config.Module, registr
121121
registry.MustRegister(isSSLGauge)
122122
registry.MustRegister(statusCodeGauge)
123123
registry.MustRegister(healthCheckResponseGaugeVec)
124-
registry.MustRegister(probeSSLEarliestCertExpiryGauge)
125-
registry.MustRegister(probeTLSVersion)
126-
registry.MustRegister(probeSSLLastInformation)
127124

128125
if !strings.HasPrefix(target, "http://") && !strings.HasPrefix(target, "https://") {
129126
target = "http://" + target
@@ -203,6 +200,7 @@ func ProbeGRPC(ctx context.Context, target string, module config.Module, registr
203200
if serverPeer != nil {
204201
tlsInfo, tlsOk := serverPeer.AuthInfo.(credentials.TLSInfo)
205202
if tlsOk {
203+
registry.MustRegister(probeSSLEarliestCertExpiryGauge, probeTLSVersion, probeSSLLastInformation)
206204
isSSLGauge.Set(float64(1))
207205
probeSSLEarliestCertExpiryGauge.Set(float64(getEarliestCertExpiry(&tlsInfo.State).Unix()))
208206
probeTLSVersion.WithLabelValues(getTLSVersion(&tlsInfo.State)).Set(1)

0 commit comments

Comments
 (0)