Use sha256 instead of md5 in node batcher processor

MD5 is insecure cryptographic hash functions, and are therefore unsuitable for security applications.
open-telemetry · Jul 30, 2019 · 13ab3d9 · 13ab3d9
1 parent 5f88de9
commit 13ab3d9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/processor/nodebatcher/node_batcher.go b/processor/nodebatcher/node_batcher.go
@@ -16,7 +16,7 @@ package nodebatcher
 
 import (
 	"context"
-	"crypto/md5"
+	"crypto/sha256"
 	"fmt"
 	"math/rand"
 	"sync"
@@ -112,7 +112,7 @@ func (b *batcher) ConsumeTraceData(ctx context.Context, td consumerdata.TraceDat
 }
 
 func (b *batcher) genBucketID(node *commonpb.Node, resource *resourcepb.Resource, spanFormat string) string {
-	h := md5.New()
+	h := sha256.New()
 	if node != nil {
 		nodeKey, err := proto.Marshal(node)
 		if err != nil {