achow101 changed the topic of #bitcoin-core-dev to: Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Weekly Meeting Thursday @ 16:00 UTC | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
agentcasey has quit [Ping timeout: 248 seconds]
Guest59 has joined #bitcoin-core-dev
eval-exec has joined #bitcoin-core-dev
Guest59 has quit [Quit: Client closed]
erdem has quit [Quit: ZNC 1.9.1 - https://znc.in]
kevkevin has quit [Remote host closed the connection]
jarthur has quit [Quit: jarthur]
eval-exec has quit [Remote host closed the connection]
adil has joined #bitcoin-core-dev
adil has quit [Quit: adil]
Guest36 has joined #bitcoin-core-dev
Guest36 has quit [Client Quit]
bitdex has quit [Ping timeout: 264 seconds]
kevkevin has joined #bitcoin-core-dev
eval-exec has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
robszarka has joined #bitcoin-core-dev
rszarka has quit [Ping timeout: 276 seconds]
agentcasey has joined #bitcoin-core-dev
greypw1495085 has quit [Remote host closed the connection]
greypw1495085 has joined #bitcoin-core-dev
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 244 seconds]
jonatack has joined #bitcoin-core-dev
josie has quit [Quit: ZNC 1.8.2 - https://znc.in]
josie has joined #bitcoin-core-dev
jarthur has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
mcey has quit [Remote host closed the connection]
mcey has joined #bitcoin-core-dev
aleggg has quit [Ping timeout: 252 seconds]
aleggg has joined #bitcoin-core-dev
diego has left #bitcoin-core-dev [#bitcoin-core-dev]
dviola has joined #bitcoin-core-dev
eval-exec has quit [Remote host closed the connection]
eval-exec has joined #bitcoin-core-dev
vasild has quit [Quit: leaving]
vasild has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
eval-exec has quit [Ping timeout: 246 seconds]
eval-exec has joined #bitcoin-core-dev
eval-exec has quit [Remote host closed the connection]
eval-exec has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto closed pull request #31780: MacOS updated Bitcoin-Core gui icon in accordance with Apple design docs (master...master) https://github.com/bitcoin/bitcoin/pull/31780
javi404 has quit [Remote host closed the connection]
javi404 has joined #bitcoin-core-dev
jarthur has quit [Quit: jarthur]
PaperSword has quit [Ping timeout: 260 seconds]
PaperSword has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [Closing Window]
PaperSword has quit [Ping timeout: 248 seconds]
PaperSword has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
BrandonOdiwuor has joined #bitcoin-core-dev
<bitcoin-git> [gui-qml] hebasto pushed 2 commits to main: https://github.com/bitcoin-core/gui-qml/compare/b4bebc316e76...765f6088d3c5
<bitcoin-git> gui-qml/main 4dea7c9 pablomartin4btc: qml, proxy: Allow IPv6 and move out UI validation
<bitcoin-git> gui-qml/main 765f608 Hennadii Stepanov: Merge bitcoin-core/gui-qml#430: Allow IPv6 in Proxy settings and moving va...
<bitcoin-git> [gui-qml] hebasto merged pull request #430: Allow IPv6 in Proxy settings and moving validation out from the UI into the model/ interface (main...qml-ipaddress-allow-ipv6) https://github.com/bitcoin-core/gui-qml/pull/430
pyth has quit [Remote host closed the connection]
pyth has joined #bitcoin-core-dev
PaperSword has quit [Quit: PaperSword]
BrandonOdiwuor has quit [Ping timeout: 240 seconds]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 245 seconds]
eval-exec has quit [Remote host closed the connection]
abubakarsadiq has joined #bitcoin-core-dev
eval-exec has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
eval-exec has quit [Ping timeout: 260 seconds]
flooded is now known as _flood
jonatack has quit [Ping timeout: 252 seconds]
jonatack has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] kevkevinpal opened pull request #31784: test: added additional coverage to waitforblock and waitforblockheight rpc's (master...moreTimeoutTests) https://github.com/bitcoin/bitcoin/pull/31784
bitdex has quit [Quit: = ""]
zeropoint has joined #bitcoin-core-dev
thoragh has quit [Ping timeout: 245 seconds]
jespada has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
zeropoint has quit [Quit: leaving]
<bitcoin-git> [bitcoin] Sjors opened pull request #31785: Have createNewBlock() ensure m_tip_block is set (master...2025/02/create_new_block) https://github.com/bitcoin/bitcoin/pull/31785
dviola has quit [Quit: WeeChat 4.5.1]
<bitcoin-git> [qa-assets] dergoegge merged pull request #216: add initial inputs for overflow, num3072_inv, and num3072_mul (main...main) https://github.com/bitcoin-core/qa-assets/pull/216
<bitcoin-git> [qa-assets] dergoegge pushed 2 commits to main: https://github.com/bitcoin-core/qa-assets/compare/5c026e2a97b5...ff2648d1387d
<bitcoin-git> qa-assets/main a556f1e marcofleon: add initial inputs for overflow, num3072_inv, and num3072_mul
<bitcoin-git> qa-assets/main ff2648d Niklas Gögge: Merge pull request #216 from marcofleon/main
bugs_ has joined #bitcoin-core-dev
thoragh has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] 0xf58ce opened pull request #31786: list 6467600: 334knUNdf1FSCDvrYsmK3N4uGv3nJnj7a3 (master...master) https://github.com/bitcoin/bitcoin/pull/31786
<bitcoin-git> [bitcoin] fanquake closed pull request #31786: list 6467600: 334knUNdf1FSCDvrYsmK3N4uGv3nJnj7a3 (master...master) https://github.com/bitcoin/bitcoin/pull/31786
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
mcey_ has joined #bitcoin-core-dev
mcey has quit [Ping timeout: 272 seconds]
vasild has quit [Ping timeout: 264 seconds]
vasild has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] willcl-ark opened pull request #31787: ci: run in worktrees (master...worktree-qol) https://github.com/bitcoin/bitcoin/pull/31787
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 265 seconds]
<bitcoin-git> [gui] rhysbeynon opened pull request #851: Case fix (master...Case-fix) https://github.com/bitcoin-core/gui/pull/851
<bitcoin-git> [gui] rhysbeynon closed pull request #851: Case fix (master...Case-fix) https://github.com/bitcoin-core/gui/pull/851
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
jespada has joined #bitcoin-core-dev
bugs_ has quit [Quit: Leaving]
<bitcoin-git> [gui] rhysbeynon opened pull request #852: Updated MacOS icon to more closely fit Apple's design standards (master...updated-icns) https://github.com/bitcoin-core/gui/pull/852
jonatack has joined #bitcoin-core-dev
mcey has joined #bitcoin-core-dev
Allanon has joined #bitcoin-core-dev
mcey_ has quit [Ping timeout: 244 seconds]
Allanon has quit [Client Quit]
jonatack has quit [Read error: Connection reset by peer]
<bitcoin-git> [bitcoin] Sjors closed pull request #31763: Pass custom DEP_OPTS and CONFIG_FLAGS to guix-build (master...2025/01/guix-config-flags) https://github.com/bitcoin/bitcoin/pull/31763
zeropoint has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
luke-jr has joined #bitcoin-core-dev
rszarka has joined #bitcoin-core-dev
luke-jr_ has quit [Ping timeout: 244 seconds]
luke-jr has quit [Read error: Connection reset by peer]
luke-jr has joined #bitcoin-core-dev
robszarka has quit [Ping timeout: 260 seconds]
jonatack has quit [Ping timeout: 244 seconds]
<bitcoin-git> [bitcoin] MonkeyKing44 opened pull request #31788: Corrected typos (master...fix/tps) https://github.com/bitcoin/bitcoin/pull/31788
Cory44 has joined #bitcoin-core-dev
Cory has quit [Ping timeout: 240 seconds]
Talkless has joined #bitcoin-core-dev
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
cotsuka has quit [Remote host closed the connection]
jarthur has joined #bitcoin-core-dev
cotsuka has joined #bitcoin-core-dev
robszarka has joined #bitcoin-core-dev
rszarka has quit [Ping timeout: 260 seconds]
jespada has joined #bitcoin-core-dev
Cory58 has joined #bitcoin-core-dev
Cory44 has quit [Ping timeout: 240 seconds]
jonatack has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] Hack666r opened pull request #31789: Typos fix (master...fix/errr) https://github.com/bitcoin/bitcoin/pull/31789
<bitcoin-git> [bitcoin] petryshkaCODE opened pull request #31790: Fixed typos (master...fixing/err) https://github.com/bitcoin/bitcoin/pull/31790
<lightlike> typo spam-bots?
<bitcoin-git> [bitcoin] hebasto closed pull request #31789: . (master...fix/errr) https://github.com/bitcoin/bitcoin/pull/31789
jonatack has quit [Ping timeout: 260 seconds]
jonatack has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto closed pull request #31790: . (master...fixing/err) https://github.com/bitcoin/bitcoin/pull/31790
<bitcoin-git> [bitcoin] hebasto closed pull request #31788: . (master...fix/tps) https://github.com/bitcoin/bitcoin/pull/31788
<darosior> Looks like it.
jonatack has quit [Ping timeout: 252 seconds]
Guyver2 has left #bitcoin-core-dev [Closing Window]
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
jonatack has joined #bitcoin-core-dev
<darosior> People who were around then, how did BIP113 deal with the upgrade issues mentioned in ConnectBlock? (It only check if a bad block wasn't let in using CheckBlock, not CtxCheckBlock[Header].) It seems the PR that implemented it only had a check in CtxCheckBlock() and none in ConnectBlock() so in theory an invalid block post-activation could have been
<darosior> accepted by an earlier version and then connected by a later version? https://github.com/bitcoin/bitcoin/pull/6566
<darosior> Another soft fork since the separation of CtxCheckBlock that introduced new rules there is Segwit. But for Segwit the RewindBlockIndex mechanism introduced in 6032f6930a56c107dad8f30c05fec4aab79c8c22 dealt with this.
<sipa> darosior: we have generally never implemented a "check if old accepted blocks prior to a softfork are valid post-software"; the assumption is that the active main chain is the same before and after
<sipa> for segwit things were different, because the serialization format changed, and it we needed to make sure that if someone upgraded to segwit after activation, they actually have the full block data available to serve to others
<darosior> Well there, as well as in CtxCheckBlock and CtxCheckBlockHeader
<darosior> Warning comment introduced in https://github.com/bitcoin/bitcoin/pull/11737
<sipa> darosior: i don't see how the presence of a warning invalidates anything i said? :)
<darosior> Fair enough :)
<sipa> it's a concern for sure, but apart from segwit, we've never actually implemented re-validation of old softforks
<sipa> to the best of my knowledge
<darosior> Ok, thanks
<sipa> hmm, perhaps the header-impacting aspects for BIP34, BIP65, and BIP66 (the fact that after 950 out of the past 1000 blocks signal, all blocks are required to signal) could be considered a re-validation aspect
<sipa> i'm now somewhat curious what would have happened if you upgraded to BIP66 while you were on the forked chain that it caused
<darosior> Wouldn't you just have marked the chain as invalid since the check happens in ConnectBlock() in the first place? (in scripts)
<sipa> i *just* mean the header aspect
<sipa> not the script validation rule
<sipa> my thinking is that the first time you start up with BIP66-enforcing software, you'd have a chainstate whose tip points to a block whose headers chain is invalid, in this setting
<bitcoin-git> [bitcoin] darosior opened pull request #31792: Double check all block rules in `ConnectBlock`, not only `CheckBlock` (master...2502_ctx_checks_in_connectblock) https://github.com/bitcoin/bitcoin/pull/31792
<darosior> RFC ^
Talkless has quit [Quit: Konversation terminated!]
<bitcoin-git> [qa-assets] murchandamus opened pull request #217: Add Murch’s inputs February 2025 (main...2025-02-murch-inputs) https://github.com/bitcoin-core/qa-assets/pull/217
jonatack has quit [Ping timeout: 244 seconds]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
pyth has quit [Ping timeout: 246 seconds]
pyth has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
b10c has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
b10c has joined #bitcoin-core-dev
pyth has quit [Ping timeout: 260 seconds]
pyth has joined #bitcoin-core-dev
jonatack has quit [Read error: Connection reset by peer]
mcey has quit [Ping timeout: 252 seconds]
jonatack has joined #bitcoin-core-dev
mcey has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
fearbeag has quit [Ping timeout: 252 seconds]
fearbeag has joined #bitcoin-core-dev
jarthur has quit [Quit: jarthur]
eugenesiegel has joined #bitcoin-core-dev
eugenesiegel has quit [Changing host]
eugenesiegel has joined #bitcoin-core-dev
<eugenesiegel> Would anybody know why specifically v27.0 nodes would disconnect inbound connections for not writing after 30 seconds? It doesn't happen with other versions and it happens with quite a few nodes. I didn't see anything in the changelog about any sort of write timeout being changed and I don't see anywhere in the code where a 30 second  timeout is
<eugenesiegel> used. Additionally, if I write a spam message on the connection, the node doesn't disconnect me.
<sipa> eugenesiegel: and this doesn't happen with 26.0?
<sipa> also, have you tried 28.0?
_flood has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] tdb3 closed pull request #30941: test: simplify timewarp test (master...20240921_simplify_timewarp_tests) https://github.com/bitcoin/bitcoin/pull/30941
<eugenesiegel> I couldn't reproduce locally when I checked out v27.0.  It doesn't happen with remote nodes on 26.0 or 28.0, only nodes on 27.0 from what I can tell.  It's also about 800 of these 27.0 nodes
vasild has quit [Ping timeout: 264 seconds]