Modified op-reth to always use all OP bootnodes when none are explici…

…tly provided. #14603 this is the linked git issue #14603
paradigmxyz · Feb 20, 2025 · 258a42a · 258a42a
1 parent c4066b0
commit 258a42a
Showing 1 changed file with 21 additions and 5 deletions.
diff --git a/crates/optimism/node/src/node.rs b/crates/optimism/node/src/node.rs
@@ -695,14 +695,17 @@ impl OpNetworkBuilder {
                     builder = builder.disable_discv4_discovery();
                 }
                 if !args.discovery.disable_discovery {
+                    let mut bootnodes = ctx.config().network.resolved_bootnodes()
+                    .or_else(|| ctx.chain_spec().bootnodes())
+                    .unwrap_or_default();
+
+                    if bootnodes.is_empty() {
+                    bootnodes = get_all_op_bootnodes(ctx);
+                    }
                     builder = builder.discovery_v5(
                         args.discovery.discovery_v5_builder(
                             rlpx_socket,
-                            ctx.config()
-                                .network
-                                .resolved_bootnodes()
-                                .or_else(|| ctx.chain_spec().bootnodes())
-                                .unwrap_or_default(),
+                            bootnodes,
                         ),
                     );
                 }
@@ -721,6 +724,19 @@ impl OpNetworkBuilder {
     }
 }
 
+fn get_all_op_bootnodes<Node>(ctx: &BuilderContext<Node>) -> Vec<Bootnode>
+where
+    Node: FullNodeTypes<Types: NodeTypes<ChainSpec: Hardforks>>,
+{
+    let network_name = ctx.config().chain.chain.clone().to_lowercase();
+
+    if network_name.contains("mainnet") {
+        params::V5OPBootnodes.to_vec()
+    } else {
+        params::V5OPTestnetBootnodes.to_vec()
+    }
+}
+
 impl<Node, Pool> NetworkBuilder<Node, Pool> for OpNetworkBuilder
 where
     Node: FullNodeTypes<Types: NodeTypes<ChainSpec = OpChainSpec, Primitives = OpPrimitives>>,