Merge pull request #238 from ava-labs/fix-some-logs

fix some error %s that should be %v due to possible nil error
ava-labs · Aug 16, 2022 · b9e6532 · b9e6532
2 parents 11b925f + 5119cbc
commit b9e6532
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cmd/server/server.go b/cmd/server/server.go
@@ -101,7 +101,7 @@ func serverFunc(cmd *cobra.Command, args []string) (err error) {
 		rootCancel()
 		// wait for server stop
 		waitForServerStop := <-errc
-		log.Warn("closed server; %s", waitForServerStop)
+		log.Warn("closed server: %v", waitForServerStop)
 	case serverClosed := <-errc:
 		// server already stopped here
 		_ = rootCancel

diff --git a/server/server.go b/server/server.go
@@ -186,7 +186,7 @@ func (s *server) Run(rootCtx context.Context) (err error) {
 		s.log.Warn("root context is done")
 
 		if !s.cfg.GwDisabled {
-			s.log.Warn("closed gRPC gateway server: %s", s.gwServer.Close())
+			s.log.Warn("closed gRPC gateway server: %v", s.gwServer.Close())
 			<-gwErrc
 		}
 
@@ -198,7 +198,7 @@ func (s *server) Run(rootCtx context.Context) (err error) {
 	case err = <-gRPCErrc:
 		s.log.Warn("gRPC server failed: %s", err)
 		if !s.cfg.GwDisabled {
-			s.log.Warn("closed gRPC gateway server: %s", s.gwServer.Close())
+			s.log.Warn("closed gRPC gateway server: %v", s.gwServer.Close())
 			<-gwErrc
 		}