Skip to content

Commit 9890058

Browse files
committed
Merge bitcoin/bitcoin#33723: chainparams: remove dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us
b0c7067 Remove unreliable seed from chainparams.cpp, and the associated README (SatsAndSports) Pull request description: The DNS seed `dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us.` is not returning a representative sample of bitcoin nodes. It currently returns nothing later than 28.1.0, breaching the policy. This PR removes that seed from the list of DNS seeds ### Rationale The [policy for seeds](https://github.com/bitcoin/bitcoin/blob/master/doc/dnsseed-policy.md) includes this: > The DNS seed results must consist exclusively of fairly selected and functioning Bitcoin nodes from the public network A number of comments below, in response to this PR, include apparent breaches of this policy: [1](bitcoin/bitcoin#33723 (comment)) [2](bitcoin/bitcoin#33723 (comment)), [3](bitcoin/bitcoin#33723 (comment)), in particular the first linked comment ([1](bitcoin/bitcoin#33723 (comment))) comparing the distribution at this seed to other seeds. This seed is not including anything later than 28.2.0, breaching this policy. To ensure the policy is followed, and the seeds include a representative sample of Bitcoin nodes, this PR removes this seed from the list ### Data I ran this: ``` # Get some ip address from that seed: # Repeated multiple times, to get many different IPs: dig +short dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us >> dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us # For each distinct ip gathered from the seed, get basic info about the node, including it's User Agent string: cat dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us | sort -u | while read ip; do echo ===; echo $ip; nmap -p 8333 --script bitcoin-info "$ip"; done > seed_versions.txt ``` and then summarized the agents with `egrep 'User Agent' seed_versions.txt | sort | uniq -c` and got: ``` 1 User Agent: /Satoshi:22.0.0/ 1 User Agent: /Satoshi:22.1.0/ 5 User Agent: /Satoshi:24.0.1/ 1 User Agent: /Satoshi:25.1.0/ 30 User Agent: /Satoshi:27.0.0/ 1 User Agent: /Satoshi:27.1.0/ 1 User Agent: /Satoshi:27.1.0/Knots:20240801/ 1 User Agent: /Satoshi:28.0.0/ 7 User Agent: /Satoshi:28.1.0/ 2 User Agent: /Satoshi:28.1.0/Knots:20250305/ ``` ACKs for top commit: l0rinc: reACK b0c7067 delta1: reACK bitcoin/bitcoin@b0c7067 Crypt-iQ: crACK b0c7067 laanwj: ACK b0c7067 murchandamus: ACK bitcoin/bitcoin@b0c7067 RandyMcMillan: ACK b0c7067 wiz: ACK b0c7067 dergoegge: ACK b0c7067 stickies-v: re-ACK b0c7067 mzumsande: ACK b0c7067 instagibbs: ACK b0c7067 Tree-SHA512: 7230b8dd24560ce6f8247e2e82ae7846ded8b91e230c59cc3643da3f5b9c12b5f025c1bb14490c19ca55f3794e81ce08106b31b3bf883d5c2dced05017123ac4
2 parents 9e02f78 + b0c7067 commit 9890058

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

contrib/seeds/README.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,13 @@ to addrman with).
1010

1111
Update `MIN_BLOCKS` in `makeseeds.py` and the `-m`/`--minblocks` arguments below, as needed.
1212

13-
The seeds compiled into the release are created from sipa's, achow101's and luke-jr's
13+
The seeds compiled into the release are created from sipa's and achow101's
1414
DNS seed, virtu's crawler, and asmap community AS map data. Run the following commands
1515
from the `/contrib/seeds` directory:
1616

1717
```
1818
curl https://bitcoin.sipa.be/seeds.txt.gz | gzip -dc > seeds_main.txt
1919
curl https://21.ninja/seeds.txt.gz | gzip -dc >> seeds_main.txt
20-
curl https://luke.dashjr.org/programs/bitcoin/files/charts/seeds.txt >> seeds_main.txt
2120
curl https://mainnet.achownodes.xyz/seeds.txt.gz | gzip -dc >> seeds_main.txt
2221
curl https://signet.achownodes.xyz/seeds.txt.gz | gzip -dc > seeds_signet.txt
2322
curl https://testnet.achownodes.xyz/seeds.txt.gz | gzip -dc > seeds_test.txt

src/kernel/chainparams.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ class CMainParams : public CChainParams {
147147
// release ASAP to avoid it where possible.
148148
vSeeds.emplace_back("seed.bitcoin.sipa.be."); // Pieter Wuille, only supports x1, x5, x9, and xd
149149
vSeeds.emplace_back("dnsseed.bluematt.me."); // Matt Corallo, only supports x9
150-
vSeeds.emplace_back("dnsseed.bitcoin.dashjr-list-of-p2p-nodes.us."); // Luke Dashjr
151150
vSeeds.emplace_back("seed.bitcoin.jonasschnelli.ch."); // Jonas Schnelli, only supports x1, x5, x9, and xd
152151
vSeeds.emplace_back("seed.btc.petertodd.net."); // Peter Todd, only supports x1, x5, x9, and xd
153152
vSeeds.emplace_back("seed.bitcoin.sprovoost.nl."); // Sjors Provoost

0 commit comments

Comments
 (0)