update peer host to parse dns instead of ip

[ASuciuX]

• fixes burnchain connection retry should periodically perform another DNS lookup #4657

Following the flow of the self.connect() function you can see that it is trying to open a TCP connection with net::TcpStream::connect((self.config.peer_host.as_str(), self.config.peer_port). If the peer_host is a DNS, it automatically tries to connect to the IPs paired to the DNS.

   #[stable(feature = "rust1", since = "1.0.0")]
    pub fn connect<A: ToSocketAddrs>(addr: A) -> io::Result<TcpStream> {
        super::each_addr(addr, net_imp::TcpStream::connect).map(TcpStream)
    }

The error was occurring because the DNS was converted directly into ipv4 address at the moment of reading the data from the config file. Afterwards, it didn’t have the DNS linked as it was only keeping the ipv4 address.

I've run the integration test suite and all of them passed. Please let me know if there is anything else I could run to check this.

❯ BITCOIND_TEST=1 RUST_BACKTRACE=full cargo nextest run --test-threads 1 --package stacks-node

So the solution applied here is to keep the DNS instead of converting it to ipv4 and it will automatically reconnect successfully.

[wileyj]

ahh just notifed this @ASuciuX but there is a merge conflict that needs to be resolved.

[ASuciuX]

Thank you @wileyj. The file was moved from stacks-node to stackslib, I've updated it.

[blockstack-devops]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.