fix: disable pubsub by default on ipfs node

Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>

pkg/ipfsutil/mobile.go

@@ -54,6 +54,16 @@ func (o *MobileOptions) fillDefault() {
 	if o.IpfsConfigPatch == nil {
 		o.IpfsConfigPatch = defaultIpfsConfigPatch
 	}
+
+	// apply default extras
+	if o.ExtraOpts == nil {
+		o.ExtraOpts = make(map[string]bool)
+	}
+
+	//  if not set, disable pubsub by default to avoid collision
+	if _, ok := o.ExtraOpts["pubsub"]; !ok {
+		o.ExtraOpts["pubsub"] = false
+	}
 }
 
 func NewIPFSMobile(ctx context.Context, repo *ipfs_mobile.RepoMobile, opts *MobileOptions) (*ipfs_mobile.IpfsMobile, error) {

service.go

@@ -171,11 +171,7 @@ func (opts *Opts) applyDefaults(ctx context.Context) error {
 		}
 
 		mrepo := ipfs_mobile.NewRepoMobile("", repo)
-		mnode, err = ipfsutil.NewIPFSMobile(ctx, mrepo, &ipfsutil.MobileOptions{
-			ExtraOpts: map[string]bool{
-				"pubsub": false,
-			},
-		})
+		mnode, err = ipfsutil.NewIPFSMobile(ctx, mrepo, &ipfsutil.MobileOptions{})
 		if err != nil {
 			return err
 		}

service_client.go

@@ -99,11 +99,7 @@ func NewPersistentServiceClient(path string) (ServiceClient, error) {
 	}
 
 	mrepo := ipfs_mobile.NewRepoMobile(path, repo)
-	mnode, err := ipfsutil.NewIPFSMobile(context.TODO(), mrepo, &ipfsutil.MobileOptions{
-		ExtraOpts: map[string]bool{
-			"pubsub": true,
-		},
-	})
+	mnode, err := ipfsutil.NewIPFSMobile(context.TODO(), mrepo, &ipfsutil.MobileOptions{})
 	if err != nil {
 		return nil, err
 	}