Merge bitcoin/bitcoin#29649: netbase: remove unnecessary log message

c70e4fc9a3 netbase: remove unnecessary log message (Matthew Zipkin)

Pull request description:

  This is a follow-up to #27375 that removes a spammy non-debug-level log message we don't need.
  See https://github.com/bitcoin/bitcoin/pull/27375#issuecomment-1994498888

ACKs for top commit:
  fanquake:
    ACK c70e4fc9a3 - thanks. Merging this now because it's swamping non-debug logs. i.e:

Tree-SHA512: 837682860abdf740fce5dc88c8599e066660cf16b4cab1473391eb51ad538ae52d236ecd3543df866e2a2165870397a8bf21ad9f5125ed8212a3fe207d615553

src/netbase.cpp

@@ -624,7 +624,6 @@ std::unique_ptr<Sock> ConnectDirectly(const CService& dest, bool manual_connecti
     }
 
     if (!ConnectToSocket(*sock, (struct sockaddr*)&sockaddr, len, dest.ToStringAddrPort(), manual_connection)) {
-        LogPrintf("Cannot connect to socket for %s\n", dest.ToStringAddrPort());
         return {};
     }