[network] re-activate message signing & verification

ensure signerIDs are populated in pubSub tests
onflow · Aug 12, 2021 · 155df69 · 155df69
1 parent 70fa19e
commit 155df69
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/network/p2p/libp2pNode.go b/network/p2p/libp2pNode.go
@@ -56,10 +56,10 @@ func DefaultLibP2PNodeFactory(ctx context.Context, log zerolog.Logger, me flow.I
 
 	// create PubSub options for libp2p to use
 	psOptions := []pubsub.Option{
-		// skip message signing
-		pubsub.WithMessageSigning(false),
-		// skip message signature
-		pubsub.WithStrictSignatureVerification(false),
+		// Use message signing
+		pubsub.WithMessageSigning(true),
+		// Use message signature
+		pubsub.WithStrictSignatureVerification(true),
 		// set max message size limit for 1-k PubSub messaging
 		pubsub.WithMaxMessageSize(maxPubSubMsgSize),
 		// no discovery

diff --git a/network/test/testUtil.go b/network/test/testUtil.go
@@ -247,9 +247,9 @@ func generateLibP2PNode(t *testing.T,
 
 	// create PubSub options for libp2p to use
 	psOptions := []pubsub.Option{
-		// skip message signing
-		pubsub.WithMessageSigning(false),
-		// skip message signature
+		// with message signing
+		pubsub.WithMessageSigning(true),
+		// with no message signature verification
 		pubsub.WithStrictSignatureVerification(false),
 		// set max message size limit for 1-k PubSub messaging
 		pubsub.WithMaxMessageSize(p2p.DefaultMaxPubSubMsgSize),