Remove DNS Seeds run by entities which were never well-established. #4671

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:master changing 2 files +3 −1
  1. TheBlueMatt commented at 4:16 AM on August 10, 2014: member

    As there is a clear policy DNS Seeds should be adhering to, they also need to be run by community members which are semi-trusted to act according to this policy. These two nodes were submitted without any clear evidence of community trust in their organizations and, as such, should either provide such evidence, or be removed. @ayeowch and @bitkevin were the original submitters of the pull requests which added these nodes. Note that they are both part of projects to add more full nodes to the network, which they can continue to do with or without a DNS Seed.

    Also, blog.open-nodes.org seems to indicate they are running nodes with aggressive outbound connection counts, amounting to attacking the network and replacing diversely-operated nodes with ones they run :(.

  2. ayeowch commented at 4:39 AM on August 10, 2014: contributor

    Addition of seed.bitnodes.io was discussed and ACK'ed in #3875. This is part of getaddr.bitnodes.io infrastructure sponsored by the Bitcoin Foundation and operated independently by myself. I believe most of the seeders are operated based on sipa's bitcoin seeder (https://github.com/sipa/bitcoin-seeder). seed.bitnodes.io is based on a separate Python implementation available from https://github.com/ayeowch/bitnodes/blob/master/seeder.py. Nodes with most common height are selected on random basis and served via NSD authoritative-only DNS server. The seeder is currently hosted on a dedicated server by Hetzner in Germany and has been one of the most healthy according to http://bitcoinstats.com/network/dns-servers/. I'd be happy to provide more evidence if so required in order to continue the operation of the seeder.

  3. TheBlueMatt commented at 4:48 AM on August 10, 2014: member

    I'm really only concerned with who the people are who are running the infrastructure. They need to be established within the community. ie before bitnodes and open-nodes they were run by developers who had been working on Bitcoin Core and had established trust that they were both technically competent and were acting for the good of Bitcoin. Though I dont propose that only developers can run a DNS seed, running a DNS seed is not something that someone who has no previous experience in the Bitcoin development community can do.

  4. TheBlueMatt commented at 4:52 AM on August 10, 2014: member

    Of course I appreciate everything people are trying to do, and though running a seed is useful, its not the most critical thing for the network, especially after #4559.

  5. mikehearn commented at 9:52 AM on August 11, 2014: contributor

    I don't think Bitcoin Core needs to care overly much about this given that it has relatively little dependence on the seeds. SPV clients rely on the seeds more, but I'm personally satisfied by Addy's contributions at least. The other one (opennodes) I haven't seen and isn't in bitcoinj: I have no opinion on that one. But my gut sense is: more is better.

  6. laanwj added the label P2P on Aug 13, 2014
  7. laanwj commented at 12:15 PM on August 13, 2014: member

    "must be run by entities which have some minimum level of trust within the Bitcoin community" I'm not sure how to judge this. But I agree with @mikehearn here, it seems bitnodes.io gets closer to it than open-nodes.org.

  8. laanwj commented at 9:54 AM on August 19, 2014: member

    So - let's move this along and remove open-nodes.org.

  9. Remove DNS Seeds run by entities which were never well-established. 550d4fa7a7
  10. TheBlueMatt force-pushed on Sep 3, 2014
  11. TheBlueMatt commented at 7:42 AM on September 3, 2014: member

    Fair enough, removed only open-nodes.org as bitnodes is only not well established to those who weren't paying attention while they were getting established.

  12. laanwj commented at 7:49 AM on September 3, 2014: member

    ACK

  13. BitcoinPullTester commented at 8:04 AM on September 3, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4671_550d4fa7a77c9763d4de9e0c0c48bc2655a65017/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  14. laanwj merged this on Sep 3, 2014
  15. laanwj closed this on Sep 3, 2014

  16. laanwj referenced this in commit 0e66f1ab26 on Sep 3, 2014
  17. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me