No description provided.
Add x9 service bit support to dnsseed.bluematt.me #8940
pull TheBlueMatt wants to merge 3 commits into bitcoin:master from TheBlueMatt:2016-10-dnsseed changing 2 files +6 −4-
TheBlueMatt commented at 3:26 PM on October 17, 2016: member
- MarcoFalke added the label Needs backport on Oct 17, 2016
- MarcoFalke added this to the milestone 0.13.1 on Oct 17, 2016
- MarcoFalke added the label P2P on Oct 17, 2016
-
MarcoFalke commented at 4:32 PM on October 17, 2016: member
ACK e5f8b0d
-
in src/chainparams.cpp:None in e5f8b0dc18 outdated
112 | @@ -113,12 +113,13 @@ class CMainParams : public CChainParams { 113 | assert(consensus.hashGenesisBlock == uint256S("0x000000000019d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f")); 114 | assert(genesis.hashMerkleRoot == uint256S("0x4a5e1e4baab89f3a32518a88c31bc87f618f76673e2cc77ab2127b7afdeda33b")); 115 | 116 | - vSeeds.push_back(CDNSSeedData("bitcoin.sipa.be", "seed.bitcoin.sipa.be", true)); // Pieter Wuille 117 | - vSeeds.push_back(CDNSSeedData("bluematt.me", "dnsseed.bluematt.me")); // Matt Corallo 118 | + // Note that of those with the service bits flag, most only support a subset of possible options 119 | + vSeeds.push_back(CDNSSeedData("bitcoin.sipa.be", "seed.bitcoin.sipa.be", true)); // Pieter Wuille, only supports x1, x5, x9, and x13
MarcoFalke commented at 4:33 PM on October 17, 2016:Are you sure @sipa supports x13. (Right now I don't get an answer)
sipa commented at 5:30 PM on October 17, 2016:Yes. There are just no matches.
TheBlueMatt commented at 5:28 PM on October 17, 2016: memberIncluded #8941.
Comment that most dnsseeds only support some service bits combos 504c72ad34Add x9 service bit support to dnsseed.bluematt.me ffb47139202449e12681My DNS seed supports filtering
Currently supports filtering by any combination of the 4 least significant bits.
TheBlueMatt force-pushed on Oct 17, 2016laanwj commented at 5:36 PM on October 17, 2016: memberutACK 2449e12
MarcoFalke commented at 5:37 PM on October 17, 2016: memberutACK 2449e12
btcdrak commented at 5:37 PM on October 17, 2016: contributorutACK 2449e12681ab549695f143d9b3ad2c24be03a78d
laanwj merged this on Oct 17, 2016laanwj closed this on Oct 17, 2016laanwj referenced this in commit 763828df49 on Oct 17, 2016sipa commented at 5:38 PM on October 17, 2016: memberutACK
laanwj referenced this in commit 9aa0c15972 on Oct 17, 2016laanwj referenced this in commit 3d770a88d3 on Oct 17, 2016laanwj referenced this in commit 5b4192bc4c on Oct 17, 2016jonasschnelli commented at 6:23 AM on October 18, 2016: contributorpost merge ACK
laanwj removed the label Needs backport on Oct 18, 2016DrahtBot 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 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
More mirrored repositories can be found on mirror.b10c.me