From 30dfb4b933a50fd366d7ed36ed4f71dbba2d382e Mon Sep 17 00:00:00 2001 From: Jille Timmermans Date: Thu, 8 Jul 2021 19:06:55 +0200 Subject: [PATCH] binarylog: Don't continue after failing to marshal the proto (#4582) --- internal/binarylog/sink.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/internal/binarylog/sink.go b/internal/binarylog/sink.go index b6881fffd59..2ae71da3e88 100644 --- a/internal/binarylog/sink.go +++ b/internal/binarylog/sink.go @@ -69,7 +69,8 @@ type writerSink struct { func (ws *writerSink) Write(e *pb.GrpcLogEntry) error { b, err := proto.Marshal(e) if err != nil { - grpclogLogger.Infof("binary logging: failed to marshal proto message: %v", err) + grpclogLogger.Errorf("binary logging: failed to marshal proto message: %v", err) + return err } hdr := make([]byte, 4) binary.BigEndian.PutUint32(hdr, uint32(len(b)))