diff options
author | Russell Yanofsky <russ@chaincode.com> | 2016-10-26 14:53:18 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2016-11-20 15:25:34 +0100 |
commit | e8461666ecd7f0f89fba92a377c0f8beffa233e9 (patch) | |
tree | 6c40073e479480f2e21d30bfa762095d50510e72 /src/test/uint256_tests.cpp | |
parent | 2cad5db6f752ad8fa2d047b67a137de76eb9c982 (diff) | |
download | bitcoin-e8461666ecd7f0f89fba92a377c0f8beffa233e9.tar.xz |
Modify getblocktxn handler not to drop requests for old blocks
The current getblocktxn implementation drops and ignores requests for old
blocks, which causes occasional sync_block timeouts during the
p2p-compactblocks.py test as reported in
https://github.com/bitcoin/bitcoin/issues/8842.
The p2p-compactblocks.py test setup creates many new blocks in a short
period of time, which can lead to getblocktxn requests for blocks below the
hardcoded depth limit of 10 blocks. This commit changes the getblocktxn
handler not to ignore these requests, so the peer nodes in the test setup
will reliably be able to sync.
The protocol change is documented in BIP-152 update "Allow block responses
to getblocktxn requests" at https://github.com/bitcoin/bips/pull/469.
The protocol change is not expected to affect nodes running outside the test
environment, because there shouldn't normally be lots of new blocks being
rapidly added that need to be synced.
Github-Pull: #9058
Rebased-From: dac53b58b555183ccc0d5e64c428528267cd98b3
Github-Pull: #9160
Rebased-From: ec34648766c4052816e4072cc61ad429430bcfd9
Diffstat (limited to 'src/test/uint256_tests.cpp')
0 files changed, 0 insertions, 0 deletions