From dc94adbbee66b294e29953b35d24a1abae2f79e3 Mon Sep 17 00:00:00 2001
From: Pavel Tumik <18602811+sagor999@users.noreply.github.com>
Date: Thu, 14 Jul 2022 19:14:52 +0000
Subject: [PATCH] [baseserver] do not log grpc messages unless they contain an
 error

---
 components/common-go/baseserver/server.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/components/common-go/baseserver/server.go b/components/common-go/baseserver/server.go
index c1e621850d..a8cd4209f9 100644
--- a/components/common-go/baseserver/server.go
+++ b/components/common-go/baseserver/server.go
@@ -269,6 +269,10 @@ func (s *Server) initializeGRPC() error {
 	unary := []grpc.UnaryServerInterceptor{
 		grpc_logrus.UnaryServerInterceptor(s.Logger(),
 			grpc_logrus.WithDecider(func(fullMethodName string, err error) bool {
+				// Skip logs for anything that does not contain an error.
+				if err == nil {
+					return false
+				}
 				// Skip gRPC healthcheck logs, they are frequent and pollute our logging infra
 				return fullMethodName != "/grpc.health.v1.Health/Check"
 			}),