From 489accbc90dd88807426c5ad6751f9a53addd754 Mon Sep 17 00:00:00 2001 From: Nico Flaig Date: Thu, 10 Oct 2024 17:28:30 +0100 Subject: [PATCH] fix: allow to disable peer discovery / discv5 (#7144) * fix: allow to disable peer discovery / discv5 * Explicitly check for false --- packages/cli/src/cmds/beacon/handler.ts | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/packages/cli/src/cmds/beacon/handler.ts b/packages/cli/src/cmds/beacon/handler.ts index d51af66e37d0..81536d7c7b41 100644 --- a/packages/cli/src/cmds/beacon/handler.ts +++ b/packages/cli/src/cmds/beacon/handler.ts @@ -175,14 +175,6 @@ export async function beaconHandlerInit(args: BeaconArgs & GlobalArgs) { // Add detailed version string for API node/version endpoint beaconNodeOptions.set({api: {commit, version}}); - // Combine bootnodes from different sources - const bootnodes = (beaconNodeOptions.get().network?.discv5?.bootEnrs ?? []).concat( - args.bootnodesFile ? readBootnodes(args.bootnodesFile) : [], - isKnownNetworkName(network) ? await getNetworkBootnodes(network) : [] - ); - // Deduplicate and set combined bootnodes - beaconNodeOptions.set({network: {discv5: {bootEnrs: [...new Set(bootnodes)]}}}); - // Set known depositContractDeployBlock if (isKnownNetworkName(network)) { const {depositContractDeployBlock} = getNetworkData(network); @@ -191,8 +183,19 @@ export async function beaconHandlerInit(args: BeaconArgs & GlobalArgs) { const logger = initLogger(args, beaconPaths.dataDir, config); const {peerId, enr} = await initPeerIdAndEnr(args, beaconPaths.beaconDir, logger); - // Inject ENR to beacon options - beaconNodeOptions.set({network: {discv5: {enr: enr.encodeTxt(), config: {enrUpdate: !enr.ip && !enr.ip6}}}}); + + if (args.discv5 !== false) { + // Inject ENR to beacon options + beaconNodeOptions.set({network: {discv5: {enr: enr.encodeTxt(), config: {enrUpdate: !enr.ip && !enr.ip6}}}}); + + // Combine bootnodes from different sources + const bootnodes = (beaconNodeOptions.get().network?.discv5?.bootEnrs ?? []).concat( + args.bootnodesFile ? readBootnodes(args.bootnodesFile) : [], + isKnownNetworkName(network) ? await getNetworkBootnodes(network) : [] + ); + // Deduplicate and set combined bootnodes + beaconNodeOptions.set({network: {discv5: {bootEnrs: [...new Set(bootnodes)]}}}); + } if (args.disableLightClientServer) { beaconNodeOptions.set({chain: {disableLightClientServer: true}});