Remove dnsseed.bitcoin.dashjr.org temporarily #29149

pull 1440000bytes wants to merge 1 commits into bitcoin:master from 1440000bytes:luke-dns changing 1 files +0 −1
  1. 1440000bytes commented at 9:28 pm on December 28, 2023: none

    Rationale:

    • Only this seeder is giving different results that include older versions: #29145#pullrequestreview-1797445282

    • Warnings on luke.dashjr.org reduces the confidence to use this as DNS seed in bitcoin core: https://pastebin.com/raw/Cwk2a1xr

    • He is not following DNS seed policy rule 0 and 1

  2. remove dnsseed.bitcoin.dashjr.org baafd62648
  3. DrahtBot commented at 9:28 pm on December 28, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #29145 (Change Luke Dashjr seed to dashjr-list-of-p2p-nodes-maybe-malware.us by luke-jr)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. DrahtBot added the label CI failed on Dec 28, 2023
  5. luke-jr commented at 1:05 am on December 29, 2023: member
    This PR is just trolling and should be closed/deleted.
  6. 1440000bytes commented at 1:29 am on December 29, 2023: none

    This PR is just trolling and should be closed/deleted.

    So you are not willing to answer questions about your DNS seed and think its hardcoded forever?

  7. 1440000bytes commented at 1:49 am on December 29, 2023: none

    BTW I am working on a project to monitor DNS seeds for different things. Last time I enquired about DNS seeds, they asked me to use seednode.

    This time Peter Todd told me there are some trade-offs. So 60% nodes are working on those trade-offs.

    Maybe the real reason is power you feel by seeding all the bitcoin network or misleading them with seed in your case.

  8. luke-jr commented at 2:19 am on December 29, 2023: member
    You have asked no questions, only posted false accusations.
  9. mzumsande commented at 2:47 am on December 29, 2023: contributor

    I don’t think a removal is warranted, just fix (or explain if there is a legit reason) the issue with only returinng old nodes.

    There is the monitoring site at https://www.21.ninja/dns-seeds/ run by @virtu - maybe having some additional statistics about the diversity of results could have caught this issue?

  10. 1440000bytes commented at 3:12 am on December 29, 2023: none

    I don’t think a removal is warranted, just fix (or explain if there is a legit reason) the issue with only returinng old nodes.

    There is the monitoring site at https://www.21.ninja/dns-seeds/ run by @virtu - maybe having some additional statistics about the diversity of results could have caught this issue?

    It just covers “count” and “share”. There are lot of things you monitor for DNS seeds. You know better than me.

    I don’t think a removal is warranted, just fix (or explain if there is a legit reason) the issue with only returinng old nodes.

    Removal is warranted as explained in PR description or we need to change policy doc for Luke Dasshjr. Further, he is not ready to answer to explain.

  11. 1440000bytes commented at 3:22 am on December 29, 2023: none

    You have asked no questions, only posted false accusations.

    1. Why seeder returns IP address of nodes with user agent that look like old nodes?

    2. If warnings on your server are true, why should we trust your domain will resolve to “honest” bitcoin nodes?

    3. Are you sure that server used for DNS seed is secure?

    4. What is the probability that US government agencies wont work with you and affect this last point? Then you spread something on social media about your biased opinion with 0.1 % hashrate?

    By downloading anything from my personal web directory here, you assume responsibility for ensuring the copy is in compliance with the laws of your jurisdiction, my jurisdiction (Florida, USA), and my server’s jurisdiction (New York, USA). If you do not agree to these terms, you are not authorized to access this website. Accessing this website implies you have accepted this agreement as a binding contract.

  12. luke-jr commented at 4:20 am on December 29, 2023: member

    just fix (or explain if there is a legit reason) the issue with only returinng old nodes.

    They weren’t old nodes when that change was made - they were the latest, intentionally excluding old nodes which didn’t enforce Taproot.

    I already updated it last night after you brought it to my attention.

    However, it should be noted that the nodes returned were perfectly fine, and there wasn’t actually a real issue.

    Removal is warranted as explained in PR description or we need to change policy doc for Luke Dasshjr.

    The PR description is pure lies. You are just outing yourself as a bad actor.

  13. 1440000bytes commented at 5:09 am on December 29, 2023: none

    However, it should be noted that the nodes returned were perfectly fine, and there wasn’t actually a real issue.

    They were missing a few important bug fixes

    The PR description is pure lies. You are just outing yourself as a bad actor.

    You wanted questions in #29149 (comment) which I shared in #29149 (comment) and are not answered yet

  14. 1440000bytes commented at 7:09 am on December 29, 2023: none
    In case its not clear many new devs/users: this is not reddit and up/down vote will be meaningless. Either PR gets merged or not and maintainer decide it.
  15. Sjors commented at 10:03 am on December 29, 2023: member

    Keep in mind that when a node gets a list of IP addresses from a DNS seed, the first thing it will do upon connection is to ask more peers. While the seed returns about a dozen addresses per query, these initial peers typically return a thousand each. So if subsequent outbound connections are randomly drawn, the odds of connecting to one of the peers initially returned by the seed are quite small.

    So it doesn’t matter that much what features these initial peers support, as long as they can give us more peers.

  16. achow101 closed this on Dec 29, 2023

  17. 1440000bytes commented at 12:56 pm on December 29, 2023: none
    @achow101 why is this pull request closed?
  18. benpbolton commented at 2:27 pm on December 29, 2023: none

    So it doesn’t matter that much what features these initial peers support, as long as they can give us more peers.

    But this seeder was

    intentionally excluding old nodes which didn’t enforce Taproot.

    and did so in a manner that was static (used a filtering method at the time that wasn’t updated) and the DNS seed policy states …

    …the results may be randomized but must not single-out any group of hosts to receive different results unless due to an urgent technical necessity and disclosed.

    So in summary by self admission, this was:

    undisclosed filtering ∧ not a technical necessity ∴ breaking DNS seed policy rules

    He has not yet addressed the hacking/compromise concerns yet, either.

    Screenshot 2023-12-29 at 7 21 48 AM

    This PR is far from ‘pure lies’. It should not have been closed without justification. I appreciate that luke has corrected the old static rules, but there are remaining concerns that need addressed.

  19. achow101 commented at 2:36 pm on December 29, 2023: member
    The issue with old nodes was resolved. There is no evidence otherwise that the seeder violates the DNS seed policy.
  20. etfmoon commented at 3:14 pm on December 29, 2023: none

    #29149 (comment)

    A DNS seed operating organization or person is expected to follow good host security practices The DNS seed results must consist exclusively of fairly selected and functioning Bitcoin nodes from the public network to the best of the operator’s understanding and capability.

    Evidence: https://pastebin.com/raw/Cwk2a1xr

    Maybe read descriptions and https://github.com/bitcoin/bitcoin/blob/master/doc/dnsseed-policy.md before close pull request

  21. 1440000bytes commented at 3:28 pm on December 29, 2023: none
    Do not close PRs. If you believe a PR should not need more research, then you can comment.
  22. achow101 commented at 4:02 pm on December 29, 2023: member

    #29149 (comment)

    A DNS seed operating organization or person is expected to follow good host security practices The DNS seed results must consist exclusively of fairly selected and functioning Bitcoin nodes from the public network to the best of the operator’s understanding and capability.

    Evidence: https://pastebin.com/raw/Cwk2a1xr

    Maybe read descriptions and https://github.com/bitcoin/bitcoin/blob/master/doc/dnsseed-policy.md before close pull request

    A server can be compromised while following security best practices. They are not foolproof, particularly to very targeted attacks. While it is concerning that the server was compromised in the past, there is no evidence that it continues to be compromised.

    AFAIK Luke has maintained control over it, continues to follow security best practices, and has not transferred the seed to anyone else. The seeder continues to behave honestly - it returns random nodes, and the vast majority of them also appear in bitcoin.sipa.be/seeds.txt.gz so can be reasonably assumed to be okay.

  23. 1440000bytes commented at 4:22 pm on December 29, 2023: none

    image

    image

  24. 1440000bytes commented at 4:24 pm on December 29, 2023: none
    Why not remove that policy doc that this point? bunch of hypocrites
  25. luke-jr commented at 5:21 pm on December 29, 2023: member

    …the results may be randomized but must not single-out any group of hosts to receive different results unless due to an urgent technical necessity and disclosed.

    To be clear, this refers to giving different results to different requesters. It does not forbid selection of which peers to return as results to everyone, which is quite normal for DNS seeds.

  26. etfmoon commented at 5:34 pm on December 29, 2023: none

    To be clear

    This pull request was closed without any answers and we know the politics involved in it with reasons.

  27. virtu commented at 2:26 pm on February 14, 2024: contributor

    There is the monitoring site at https://www.21.ninja/dns-seeds/ run by @virtu - maybe having some additional statistics about the diversity of results could have caught this issue?

    Finally got around to this. I added two:

  28. bitcoin locked this on Feb 13, 2025

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: 2025-11-02 21:13 UTC

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