diff options
author | Jim Posen <jim.posen@gmail.com> | 2020-05-04 11:13:13 -0400 |
---|---|---|
committer | John Newbery <john@johnnewbery.com> | 2020-05-31 23:01:06 -0400 |
commit | 132b30d9c84f2a8053714a438f227b583a89a9ea (patch) | |
tree | fbabe20e488f17a3d1d58724d772d5381723b256 /src | |
parent | b3fbc94d4f2937bb682f2766cc9a8d4fde328a3f (diff) | |
download | bitcoin-132b30d9c84f2a8053714a438f227b583a89a9ea.tar.xz |
[net] Signal NODE_COMPACT_FILTERS if we're serving compact filters.
If -peerblockfilters is configured, signal the NODE_COMPACT_FILTERS service
bit to indicate that we are able to serve compact block filters, headers
and checkpoints.
Diffstat (limited to 'src')
-rw-r--r-- | src/init.cpp | 12 | ||||
-rw-r--r-- | src/net_processing.cpp | 2 | ||||
-rw-r--r-- | src/protocol.cpp | 1 | ||||
-rw-r--r-- | src/protocol.h | 3 |
4 files changed, 12 insertions, 6 deletions
diff --git a/src/init.cpp b/src/init.cpp index 37e6251295..24ddaac066 100644 --- a/src/init.cpp +++ b/src/init.cpp @@ -996,11 +996,13 @@ bool AppInitParameterInteraction() } } - // Basic filters are the only supported filters. The basic filters index must be enabled - // to serve compact filters - if (gArgs.GetBoolArg("-peerblockfilters", DEFAULT_PEERBLOCKFILTERS) && - g_enabled_filter_types.count(BlockFilterType::BASIC) != 1) { - return InitError(_("Cannot set -peerblockfilters without -blockfilterindex.")); + // Signal NODE_COMPACT_FILTERS if peerblockfilters and basic filters index are both enabled. + if (gArgs.GetBoolArg("-peerblockfilters", DEFAULT_PEERBLOCKFILTERS)) { + if (g_enabled_filter_types.count(BlockFilterType::BASIC) != 1) { + return InitError(_("Cannot set -peerblockfilters without -blockfilterindex.")); + } + + nLocalServices = ServiceFlags(nLocalServices | NODE_COMPACT_FILTERS); } // if using block pruning, then disallow txindex diff --git a/src/net_processing.cpp b/src/net_processing.cpp index 321e54b3dc..587b655d1c 100644 --- a/src/net_processing.cpp +++ b/src/net_processing.cpp @@ -2009,7 +2009,7 @@ static bool PrepareBlockFilterRequest(CNode& peer, const CChainParams& chain_par { const bool supported_filter_type = (filter_type == BlockFilterType::BASIC && - gArgs.GetBoolArg("-peerblockfilters", DEFAULT_PEERBLOCKFILTERS)); + (peer.GetLocalServices() & NODE_COMPACT_FILTERS)); if (!supported_filter_type) { LogPrint(BCLog::NET, "peer %d requested unsupported block filter type: %d\n", peer.GetId(), static_cast<uint8_t>(filter_type)); diff --git a/src/protocol.cpp b/src/protocol.cpp index 83a24b9d95..7f58125f00 100644 --- a/src/protocol.cpp +++ b/src/protocol.cpp @@ -213,6 +213,7 @@ static std::string serviceFlagToStr(size_t bit) case NODE_GETUTXO: return "GETUTXO"; case NODE_BLOOM: return "BLOOM"; case NODE_WITNESS: return "WITNESS"; + case NODE_COMPACT_FILTERS: return "COMPACT_FILTERS"; case NODE_NETWORK_LIMITED: return "NETWORK_LIMITED"; // Not using default, so we get warned when a case is missing } diff --git a/src/protocol.h b/src/protocol.h index 985f44640b..a68f30287d 100644 --- a/src/protocol.h +++ b/src/protocol.h @@ -285,6 +285,9 @@ enum ServiceFlags : uint64_t { // NODE_WITNESS indicates that a node can be asked for blocks and transactions including // witness data. NODE_WITNESS = (1 << 3), + // NODE_COMPACT_FILTERS means the node will service basic block filter requests. + // See BIP157 and BIP158 for details on how this is implemented. + NODE_COMPACT_FILTERS = (1 << 6), // NODE_NETWORK_LIMITED means the same as NODE_NETWORK with the limitation of only // serving the last 288 (2 day) blocks // See BIP159 for details on how this is implemented. |