constructor: always set pubsub options

License: MIT Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
ipfs · Jun 7, 2019 · dc23f54 · dc23f54
1 parent 7df0356
commit dc23f54
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/core/coreapi/node.go b/core/coreapi/node.go
@@ -457,13 +457,9 @@ func configSwarm(swarm config.SwarmConfig, exp config.Experiments) Option {
 }
 
 func configPubsub(ps config.PubsubConfig) Option {
-	var pubsubOptions []pubsub.Option
-	if ps.DisableSigning {
-		pubsubOptions = append(pubsubOptions, pubsub.WithMessageSigning(false))
-	}
-
-	if ps.StrictSignatureVerification {
-		pubsubOptions = append(pubsubOptions, pubsub.WithStrictSignatureVerification(true))
+	pubsubOptions := []pubsub.Option{
+		pubsub.WithMessageSigning(!ps.DisableSigning),
+		pubsub.WithStrictSignatureVerification(ps.StrictSignatureVerification),
 	}
 
 	switch ps.Router {
@@ -658,7 +654,6 @@ func defaults() settings {
 	out.components[ProviderKeys] = fx.Provide(simple.NewBlockstoreProvider)
 	out.components[Reprovider] = fx.Provide(node.SimpleReprovider(node.DefaultReprovideFrequency))
 
-
 	out.components[Exchange] = fx.Provide(offline.Exchange)
 	out.components[Namesys] = fx.Provide(node.Namesys(0))
 	out.components[Blockservice] = fx.Provide(node.BlockService)