[network] re-activate message signing & verification

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

Hide file tree

Showing 2 changed files with 8 additions and 8 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,10 +247,10 @@ func generateLibP2PNode(t *testing.T,
 
 	// create PubSub options for libp2p to use
 	psOptions := []pubsub.Option{
-		// skip message signing
-		pubsub.WithMessageSigning(false),
-		// skip message signature
-		pubsub.WithStrictSignatureVerification(false),
+		// with message signing
+		pubsub.WithMessageSigning(true),
+		// with no message signature verification
+		pubsub.WithStrictSignatureVerification(true),
 		// set max message size limit for 1-k PubSub messaging
 		pubsub.WithMaxMessageSize(p2p.DefaultMaxPubSubMsgSize),
 	}