perf: resolve trusted nodes concurrently (#9291)

paradigmxyz · Jul 4, 2024 · e7803f3 · e7803f3
1 parent 0373c58
commit e7803f3
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/crates/node/builder/src/launch/common.rs b/crates/node/builder/src/launch/common.rs
@@ -209,15 +209,16 @@ impl LaunchContextWith<WithConfigs> {
             info!(target: "reth::cli", "Adding trusted nodes");
 
             // resolve trusted peers if they use a domain instead of dns
-            for peer in &self.attachment.config.network.trusted_peers {
+            let resolved = futures::future::try_join_all(
+            self.attachment.config.network.trusted_peers.iter().map(|peer| async move {
                 let backoff = ConstantBuilder::default()
                     .with_max_times(self.attachment.config.network.dns_retries);
-                let resolved = (move || { peer.resolve() })
-                .retry(&backoff)
-                .notify(|err, _| warn!(target: "reth::cli", "Error resolving peer domain: {err}. Retrying..."))
-                .await?;
-                self.attachment.toml_config.peers.trusted_nodes.insert(resolved);
-            }
+                (move || { peer.resolve() })
+                    .retry(&backoff)
+                    .notify(|err, _| warn!(target: "reth::cli", "Error resolving peer domain: {err}. Retrying..."))
+                    .await
+            })).await?;
+            self.attachment.toml_config.peers.trusted_nodes.extend(resolved);
         }
         Ok(self)
     }