Fix logrus interceptor (grpc-ecosystem#588)

* interceptors: Fix logrus example to iterate over all fields Fixes grpc-ecosystem#570 Signed-off-by: Chance Zibolski <[email protected]> * interceptors: Update zap InterceptorLogger to use Iterator properly The logic in the example is redundant with the logging.Fields Iterator Signed-off-by: Chance Zibolski <[email protected]> --------- Signed-off-by: Chance Zibolski <[email protected]>
rahulkhairwar · Jul 6, 2023 · ea10178 · ea10178
1 parent e895693
commit ea10178
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/interceptors/logging/examples/logrus/example_test.go b/interceptors/logging/examples/logrus/example_test.go
@@ -18,7 +18,7 @@ func InterceptorLogger(l logrus.FieldLogger) logging.Logger {
 	return logging.LoggerFunc(func(_ context.Context, lvl logging.Level, msg string, fields ...any) {
 		f := make(map[string]any, len(fields)/2)
 		i := logging.Fields(fields).Iterator()
-		if i.Next() {
+		for i.Next() {
 			k, v := i.At()
 			f[k] = v
 		}