From bf77c863aee306764c6934d29dbde868eb953fef Mon Sep 17 00:00:00 2001 From: "Dmitriy S. Sinyavskiy" Date: Tue, 12 Jul 2022 11:50:05 +0300 Subject: [PATCH] Fix memory leakage in kit.PayloadUnaryServerInterceptor memory leakage Closes issue #498 --- logging/kit/payload_interceptors.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/logging/kit/payload_interceptors.go b/logging/kit/payload_interceptors.go index 5dc8031c8..b3c4370df 100644 --- a/logging/kit/payload_interceptors.go +++ b/logging/kit/payload_interceptors.go @@ -31,8 +31,8 @@ func PayloadUnaryServerInterceptor(logger log.Logger, decider grpc_logging.Serve return handler(ctx, req) } // Use the provided log.Logger for logging but use the fields from context. - logger = log.With(logger, append(serverCallFields(info.FullMethod), ctxkit.TagsToFields(ctx)...)...) - logProtoMessageAsJson(logger, req, "grpc.request.content", "server request payload logged as grpc.request.content field") + logEntry := log.With(logger, append(serverCallFields(info.FullMethod), ctxkit.TagsToFields(ctx)...)...) + logProtoMessageAsJson(logEntry, req, "grpc.request.content", "server request payload logged as grpc.request.content field") resp, err := handler(ctx, req) if err == nil { logProtoMessageAsJson(logger, resp, "grpc.response.content", "server response payload logged as grpc.request.content field")