Address CR comments for FAB-17000

This change set allocates a designated logger for the certificate expiration tracking, for easier log capturing. Change-Id: I83443f10f7a34153246a66fd8069f69ad01b43fe Signed-off-by: yacovm <[email protected]>
hyperledger · Nov 8, 2019 · 649d563 · 649d563
1 parent d7c5830
commit 649d563
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
@@ -276,13 +276,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)