Add DNS seed support for testnet #2237

pull petertodd wants to merge 1 commits into bitcoin:master from petertodd:testnet-dnsseed changing 1 files +26 −21

petertodd commented at 4:18 AM on January 30, 2013: contributor

As discussed on the dev mailing list. Both IRC and the DNS seed are enabled by default for testnet; mainnet is unchanged.
Add DNS seed support for testnet af8998824c
BitcoinPullTester commented at 5:07 AM on January 30, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/af8998824c7c1c762c6b47c32afefb7d1f4f0276 for binaries and test log.
in src/net.cpp:None in af8998824c
```
1153 |      {"xf2.org", "bitseed.xf2.org"},
1154 | +    {NULL, NULL}
1155 | +};
1156 | +
1157 | +static const char *strTestNetDNSSeed[][2] = {
1158 | +    {"bitcoin.petertodd.org", "testnet-seed.bitcoin.petertodd.org"},
```
Diapolo commented at 6:08 AM on January 30, 2013:

Isn't it a little problemativ, if we only have your seed addresses here? Can this be abused somehow?

petertodd commented at 6:24 AM on January 30, 2013:

Absolutely. The only reason it's acceptable at all is because it's testnet.

If anyone else wants to run a seed let me know: I can give you a already configured Amazon EC2 image ready to go.
jgarzik commented at 6:19 AM on January 30, 2013: contributor

No major objection, but it is unusual that there are two DNS seeds from one entity. Not typical policy.
petertodd commented at 6:27 AM on January 30, 2013: contributor

That's actually just one seed, look at the comment above the seed defs, src/net.cpp:1147 "The first name is used as information source for addrman."
jgarzik commented at 2:46 PM on January 30, 2013: contributor

Whoops, you're right. Comment rescinded.
jgarzik referenced this in commit 353b7f4a9c on Feb 4, 2013
jgarzik merged this on Feb 4, 2013
jgarzik closed this on Feb 4, 2013
petertodd deleted the branch on Apr 30, 2013
laudney referenced this in commit f293ef6c8c on Mar 19, 2014
DrahtBot locked this on Sep 8, 2021

Contributors