Merge "Address CR comments for FAB-17000"

hyperledger · Nov 8, 2019 · 4952829 · 4952829
2 parents 2332eea + 649d563
commit 4952829
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/internal/peer/node/start.go b/internal/peer/node/start.go
@@ -285,13 +285,13 @@ func serve(args []string) error {
 		logger.Panicf("Failed to serialize the signing identity: %v", err)
 	}
 
-	// Log warnings if some certificate (e-cert, TLS certs) expires soon
+	expirationLogger := flogging.MustGetLogger("certmonitor")
 	crypto.TrackExpiration(
 		serverConfig.SecOpts.UseTLS,
 		serverConfig.SecOpts.Certificate,
 		cs.GetClientCertificate().Certificate,
 		signingIdentityBytes,
-		logger.Warnf, // This can be used to piggyback a metric event in the future
+		expirationLogger.Warnf, // This can be used to piggyback a metric event in the future
 		time.Now(),
 		time.AfterFunc)
 

diff --git a/orderer/common/server/main.go b/orderer/common/server/main.go
@@ -163,12 +163,13 @@ func Main() {
 		logger.Panicf("Failed serializing signing identity: %v", err)
 	}
 
+	expirationLogger := flogging.MustGetLogger("certmonitor")
 	crypto.TrackExpiration(
 		serverConfig.SecOpts.UseTLS,
 		serverConfig.SecOpts.Certificate,
 		[][]byte{clusterClientConfig.SecOpts.Certificate},
 		identityBytes,
-		logger.Warnf, // This can be used to piggyback a metric event in the future
+		expirationLogger.Warnf, // This can be used to piggyback a metric event in the future
 		time.Now(),
 		time.AfterFunc)