diff options
author | Ava Chow <github@achow101.com> | 2024-08-19 15:15:02 -0400 |
---|---|---|
committer | Ava Chow <github@achow101.com> | 2024-08-19 15:15:02 -0400 |
commit | d79ea809d28197b1b4e3748aa1715272b53601d0 (patch) | |
tree | ed0a5c4994f7c399bc430db39c9921bd927917bc /src/node/psbt.cpp | |
parent | ee367170cb2acf82b6ff8e0ccdbc1cce09730662 (diff) | |
parent | 16e95bda86302af20cfb314a2c0252256d01f750 (diff) | |
download | bitcoin-master.tar.xz |
Merge bitcoin/bitcoin#30647: Move maximum timewarp attack threshold back to 600s from 7200sHEADmaster
16e95bda86302af20cfb314a2c0252256d01f750 Move maximum timewarp attack threshold back to 600s from 7200s (Matt Corallo)
Pull request description:
In 6bfa26048dbafb91e9ca63ea8d3960271e798098 the testnet4 timewarp attack fix block time variation was increased from the Great Consensus Cleanup value of 600s to 7200s on the thesis that this allows miners to always create blocks with the current time. Sadly, doing so does allow for some nonzero inflation, even if not a huge amount.
While it could be that some hardware ignores the timestamp provided to it over Stratum and forces the block header timestamp to the current time, I'm not aware of any such hardware, and it would also likely suffer from random invalid blocks due to relying on NTP anyway, making its existence highly unlikely.
This leaves the only concern being pools, but most of those rely on work generated by Bitcoin Core (in one way or another, though when spy mining possibly not), and it seems likely that they will also not suffer any lost work. While its possible that a pool does generate invalid work due to spy mining or otherwise custom logic, it seems unlikely that a substantial portion of hashrate would do so, making the difference somewhat academic (any pool that screws this up will only do so once and the network would come out just fine).
Further, while we may end up deciding these assumptions were invalid and we should instead use 7200s, it seems prudent to try with the value we "want" on testnet4, giving us the ability to learn if the compatibility concerns are an issue before we go to mainnet.
ACKs for top commit:
fjahr:
tACK 16e95bda86302af20cfb314a2c0252256d01f750
achow101:
ACK 16e95bda86302af20cfb314a2c0252256d01f750
murchandamus:
crACK 16e95bda86302af20cfb314a2c0252256d01f750
Tree-SHA512: ae46d03b728b6e23cb6ace64c9813bc01c01e38dd7f159cf0fab53b331ef84b3b811edab225453ccdfedb53b242f55b0efd69829782657490fe393d24dacbeb2
Diffstat (limited to 'src/node/psbt.cpp')
0 files changed, 0 insertions, 0 deletions