diff options
author | Martin Zumsande <mzumsande@gmail.com> | 2022-07-22 14:51:44 -0400 |
---|---|---|
committer | Martin Zumsande <mzumsande@gmail.com> | 2022-09-06 15:16:35 -0400 |
commit | 385f5a4c3feb716fcf3f2b4823535df6da6bb67b (patch) | |
tree | aa4a7998d8c5784b0164c52d31ff6da762ac18a4 /src/net.cpp | |
parent | 91f0a7fbb79fe81a75370a4b60dcdd2e55edfa81 (diff) | |
download | bitcoin-385f5a4c3feb716fcf3f2b4823535df6da6bb67b.tar.xz |
p2p: Don't query DNS seeds when both IPv4 and IPv6 are unreachable
This happens, for example, if the user specified -onlynet=onion or
-onlynet=i2p. DNS seeds only resolve to IPv4 / IPv6 addresses,
making their answers useless to us, since we don't want to make
connections to these.
If, within the DNS seed thread, we'd instead do fallback AddrFetch
connections to one of the clearnet addresses the DNS seed resolves to,
we might get usable addresses from other networks
if lucky, but would be violating our -onlynet user preference
in doing so.
Therefore, in this case it is better to rely on fixed seeds for networks we
want to connect to.
Co-authored-by: Vasil Dimov <vd@FreeBSD.org>
Diffstat (limited to 'src/net.cpp')
-rw-r--r-- | src/net.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/net.cpp b/src/net.cpp index b7d9829b13..5db1b4e1a9 100644 --- a/src/net.cpp +++ b/src/net.cpp @@ -1630,7 +1630,7 @@ void CConnman::ThreadOpenConnections(const std::vector<std::string> connect) LOCK2(m_addr_fetches_mutex, m_added_nodes_mutex); if (m_addr_fetches.empty() && m_added_nodes.empty()) { add_fixed_seeds_now = true; - LogPrintf("Adding fixed seeds as -dnsseed=0, -addnode is not provided and all -seednode(s) attempted\n"); + LogPrintf("Adding fixed seeds as -dnsseed=0 (or IPv4/IPv6 connections are disabled via -onlynet), -addnode is not provided and all -seednode(s) attempted\n"); } } |