fix(dns-seeds): use correct cloudflare resolver default (#4029)

Description
---
- changes default domain name server to `1.1.1.1:853/cloudflare-dns.com`

Motivation and Context
---
Previous default does not work (connection refused)

How Has This Been Tested?
---
Unit test that checks `seeds.weatherwax.tari.com` works
Manually, base node adds all seeds from  `seeds.weatherwax.tari.com`

base_layer/p2p/src/config.rs

@@ -20,10 +20,7 @@
 //  WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE
 //  USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-use std::{
-    path::{Path, PathBuf},
-    str::FromStr,
-};
+use std::path::{Path, PathBuf};
 
 use serde::{Deserialize, Serialize};
 use tari_common::{
@@ -37,7 +34,7 @@ use tari_common::{
 use tari_comms::multiaddr::Multiaddr;
 use tari_comms_dht::DhtConfig;
 
-use crate::transport::TransportConfig;
+use crate::{transport::TransportConfig, DEFAULT_DNS_NAME_SERVER};
 
 /// Peer seed configuration
 #[derive(Clone, Debug, Serialize, Deserialize)]
@@ -65,7 +62,7 @@ impl Default for PeerSeedsConfig {
             override_from: None,
             peer_seeds: StringList::default(),
             dns_seeds: StringList::default(),
-            dns_seeds_name_server: DnsNameServer::from_str("1.1.1.1:53/cloudflare.net").unwrap(),
+            dns_seeds_name_server: DEFAULT_DNS_NAME_SERVER.parse().unwrap(),
             dns_seeds_use_dnssec: false,
         }
     }

base_layer/p2p/src/initialization.rs

@@ -446,7 +446,17 @@ impl P2pInitializer {
             return Ok(Vec::new());
         }
 
-        debug!(target: LOG_TARGET, "Resolving DNS seeds...");
+        debug!(
+            target: LOG_TARGET,
+            "Resolving DNS seeds (NS:{}, addresses: {})...",
+            config.dns_seeds_name_server,
+            config
+                .dns_seeds
+                .iter()
+                .map(ToString::to_string)
+                .collect::<Vec<String>>()
+                .join(",")
+        );
         let start = Instant::now();
 
         let resolver = if config.dns_seeds_use_dnssec {

base_layer/p2p/src/peer_seeds.rs

@@ -236,11 +236,9 @@ mod test {
 
         #[tokio::test]
         async fn it_returns_seeds_from_real_address() {
-            let mut resolver = DnsSeedResolver {
-                client: DnsClient::connect(DEFAULT_DNS_NAME_SERVER.parse().unwrap())
-                    .await
-                    .unwrap(),
-            };
+            let mut resolver = DnsSeedResolver::connect(DEFAULT_DNS_NAME_SERVER.parse().unwrap())
+                .await
+                .unwrap();
             let seeds = resolver.resolve("seeds.weatherwax.tari.com").await.unwrap();
             println!("{:?}", seeds);
             assert!(!seeds.is_empty());