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
joetor5 has quit [Ping timeout: 276 seconds]
jespada_ has quit [Ping timeout: 252 seconds]
joetor5 has joined #bitcoin-core-dev
badkat has joined #bitcoin-core-dev
pyth has joined #bitcoin-core-dev
joetor5 has quit [Ping timeout: 265 seconds]
Cory94 has quit [Quit: Client closed]
Cory94 has joined #bitcoin-core-dev
bitdex has quit [Ping timeout: 264 seconds]
PaperSword has quit [Remote host closed the connection]
MrHAPPY has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
pyth has quit [Ping timeout: 245 seconds]
PaperSword has quit [Client Quit]
pyth has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 252 seconds]
badkat has quit [Quit: Under attack!]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin_ has quit [Remote host closed the connection]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevi__ has joined #bitcoin-core-dev
kevkevi__ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
Cory94 has quit [Quit: Client closed]
Cory94 has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
badkat has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
jon_atack has quit [Ping timeout: 265 seconds]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Write error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
cmirror has quit [Remote host closed the connection]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
cmirror has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
LainIwakura has quit [Quit: Client closed]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
MrHAPPY has quit []
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
joetor5 has quit [Quit: joetor5]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevi__ has joined #bitcoin-core-dev
kevkevi__ has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevi__ has joined #bitcoin-core-dev
kevkevi__ has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevi__ has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevi__ has quit [Read error: Connection reset by peer]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
LainIwakura has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
kevkevin_ has joined #bitcoin-core-dev
kevkevi__ has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevi__ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
LainIwakura has quit [Quit: Client closed]
kevkevin has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
Talkless has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
antanst has quit [Ping timeout: 276 seconds]
Talkless has quit [Client Quit]
kevkevin_ has joined #bitcoin-core-dev
antanst has joined #bitcoin-core-dev
kevkevin_ has quit [Remote host closed the connection]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Read error: Connection reset by peer]
kevkevin_ has quit [Read error: Connection reset by peer]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 252 seconds]
Christoph_ has joined #bitcoin-core-dev
Christoph_ has quit [Remote host closed the connection]
Christoph_ has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
letim1291 has joined #bitcoin-core-dev
letim1291 has quit [Ping timeout: 240 seconds]
Guyver2 has joined #bitcoin-core-dev
antanst4 has joined #bitcoin-core-dev
antanst has quit [Ping timeout: 276 seconds]
antanst4 is now known as antanst
Guyver2 has left #bitcoin-core-dev [Closing Window]
<bitcoin-git> [bitcoin] maflcko opened pull request #32360: test: Force named args in RPCOverloadWrapper optional args (master...2504-named-args) https://github.com/bitcoin/bitcoin/pull/32360
Christoph_ has quit [Quit: Christoph_]
j03lliott has joined #bitcoin-core-dev
j03lliott has quit [Client Quit]
j03lliott has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
j03lliott has quit [Client Quit]
storopoli has quit [Ping timeout: 272 seconds]
storopoli has joined #bitcoin-core-dev
j03lliott has joined #bitcoin-core-dev
j03lliott has quit [Ping timeout: 240 seconds]
robszarka has joined #bitcoin-core-dev
szarka has quit [Ping timeout: 260 seconds]
Christoph_ has quit [Quit: Christoph_]
Christoph_ has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 252 seconds]
neutrino777 has quit [Ping timeout: 248 seconds]
neutrino777 has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 272 seconds]
rkrux has joined #bitcoin-core-dev
sliv3r__ has quit [Ping timeout: 248 seconds]
pablomartin has quit [Ping timeout: 276 seconds]
<rkrux> hey, re this PR: #29124, it's just 1 functional test and had 3 ACKs previously and I have ACKed it again after it needed a rebase. I believe it's rfm but does it require the other reviewers to ACK again?
<corebot> https://github.com/bitcoin/bitcoin/issues/29124 | test: Test that migration automatically repairs corrupted metadata with doubled derivation path by achow101 · Pull Request #29124 · bitcoin/bitcoin · GitHub
Cory94 has quit [Ping timeout: 240 seconds]
Cory94 has joined #bitcoin-core-dev
jespada has joined #bitcoin-core-dev
rkrux has quit [Quit: Client closed]
flooded has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 276 seconds]
Christoph_ has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 248 seconds]
Christoph_ has joined #bitcoin-core-dev
Cory94 has quit [Quit: Client closed]
Cory94 has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
LainIwakura has joined #bitcoin-core-dev
leeya has joined #bitcoin-core-dev
LainIwakura has quit [Client Quit]
LainIwakura has joined #bitcoin-core-dev
LainIwakura has quit [Client Quit]
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/d62c2d82e14d...f409444d0243
<bitcoin-git> bitcoin/master f57db75 David Gumberg: build: Use `-z noseparate-code` on NetBSD < 11.0
<bitcoin-git> bitcoin/master 00ba3ba David Gumberg: build: Drop option for disabling hardening
<bitcoin-git> bitcoin/master 77e553a David Gumberg: build: refactor: hardening flags -> core_interface
<bitcoin-git> [bitcoin] fanquake merged pull request #32071: build: Drop option to disable hardening. (master...3-14-25-drop-hardening-option) https://github.com/bitcoin/bitcoin/pull/32071
LainIwakura has joined #bitcoin-core-dev
pablomartin has joined #bitcoin-core-dev
leeya has quit [Ping timeout: 240 seconds]
Christoph_ has quit [Ping timeout: 252 seconds]
Christoph_ has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
jespada_ has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 276 seconds]
robszarka has quit [Quit: Leaving]
szarka has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 260 seconds]
Guyver2 has joined #bitcoin-core-dev
sliv3r__ has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [#bitcoin-core-dev]
pablomartin has quit [Ping timeout: 248 seconds]
<bitcoin-git> [bitcoincore.org] darosior opened pull request #1127: Add security advisory for remaining CVE-2024-52919 issue (master...2503_CVE-2024-52919_again) https://github.com/bitcoin-core/bitcoincore.org/pull/1127
Cory94 has quit [Quit: Client closed]
Cory94 has joined #bitcoin-core-dev
katsu_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] 9AeinBagheri9 opened pull request #32362: Update SECURITY.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/32362
katsu has quit [Ping timeout: 248 seconds]
<bitcoin-git> [bitcoin] fanquake closed pull request #32362: Update SECURITY.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/32362
katsu_ is now known as katsu
<bitcoin-git> [bitcoin] GarmashAlex opened pull request #32363: ci: Use previous releases in tests on Windows (master...hpp1) https://github.com/bitcoin/bitcoin/pull/32363
<bitcoin-git> [bitcoin] DrahtBot closed pull request #32363: ci: Use previous releases in tests on Windows (master...hpp1) https://github.com/bitcoin/bitcoin/pull/32363
<bitcoin-git> [bitcoin] hebasto reopened pull request #32363: ci: Use previous releases in tests on Windows (master...hpp1) https://github.com/bitcoin/bitcoin/pull/32363
LainIwakura has quit [Quit: Client closed]
<hebasto> maflcko: that ^ PR looks legit
LainIwakura has joined #bitcoin-core-dev
bugs_ has joined #bitcoin-core-dev
MrHAPPY has joined #bitcoin-core-dev
leeya has joined #bitcoin-core-dev
leeya has quit [Ping timeout: 240 seconds]
<fanquake> hebasto: why? It's the same author who's been spamming crypto projects with AI generated garbage
<fanquake> The PR description alone is obviously AI
<fanquake> Half the code changes don't make snese
<maflcko> hebasto: thx. The rule was added, to close spam such as https://github.com/bitcoin/bitcoin/pull/32228/files. But I removed it in https://github.com/maflcko/DrahtBot/commit/d5ce5a7859709423422872daef6c43513bfd80f4 again.
<fanquake> maflacko: it was correct here though?
<fanquake> *maflcko
<maflcko> fanquake: Yeah, I had the impression this was AI generated, but some original thought may have gone into it
<maflcko> idk, happy to wait a day how the feedback is addressed
<maflcko> Also, could run the CI to check if its passing, as this is a ci-only change. If it isn't passing, it should be easy to reject
<fanquake> I think it's already failing multiple CIs
<maflcko> ah, yeah. It looks like a bot or "ai agent"
<bitcoin-git> [bitcoin] stickies-v opened pull request #32364: refactor: validation: mark CheckBlockIndex as const (master...2025-04/const-checkblockindex) https://github.com/bitcoin/bitcoin/pull/32364
Christoph_ has quit [Quit: Christoph_]
eugenesiegel has joined #bitcoin-core-dev
eugenesiegel has quit [Quit: Client closed]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f409444d0243...3a29ba33dca6
<bitcoin-git> bitcoin/master 35e57fb Hennadii Stepanov: depends: Fix cross-compiling `qt` package from macOS to Windows
<bitcoin-git> bitcoin/master 3a29ba3 merge-script: Merge bitcoin/bitcoin#32357: depends: Fix cross-compiling `qt` package fro...
<bitcoin-git> [bitcoin] fanquake merged pull request #32357: depends: Fix cross-compiling `qt` package from macOS to Windows (master...250426-macos-to-windows) https://github.com/bitcoin/bitcoin/pull/32357
eugenesiegel has joined #bitcoin-core-dev
PaperSword1 has joined #bitcoin-core-dev
PaperSword has quit [Ping timeout: 260 seconds]
PaperSword1 is now known as PaperSword
<bitcoin-git> [bitcoin] maflcko opened pull request #32365: descriptors: Reject + sign while parsing unsigned (master...2504-int-parsing) https://github.com/bitcoin/bitcoin/pull/32365
MrHAPPY has quit []
Cory94 has quit [Quit: Client closed]
MrHAPPY has joined #bitcoin-core-dev
Cory94 has joined #bitcoin-core-dev
eugenesiegel has quit [Quit: Client closed]
MrHAPPY has quit [Ping timeout: 265 seconds]
tinova4 has quit [Ping timeout: 248 seconds]
tinova4 has joined #bitcoin-core-dev
MrHAPPY has joined #bitcoin-core-dev
<bitcoin-git> [gui] D33r-Gee opened pull request #869: interface: Expose load utxo snapshot functionality (master...2025_gui_interfaces_loadsnapshot) https://github.com/bitcoin-core/gui/pull/869
eugenesiegel has joined #bitcoin-core-dev
sunyata has joined #bitcoin-core-dev
LainIwakura has quit [Quit: Client closed]
eugenesiegel has quit [Quit: Client closed]
eugenesiegel has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] stutxo opened pull request #32366: add ignore_incoming_blocks option to ProcessMessage and SendMessages (master...tx_only_mode) https://github.com/bitcoin/bitcoin/pull/32366
<bitcoin-git> [bitcoin] stutxo closed pull request #32366: add ignore_incoming_blocks option to ProcessMessage and SendMessages (master...tx_only_mode) https://github.com/bitcoin/bitcoin/pull/32366
neutrino777 has quit [Read error: Connection reset by peer]
neutrino777 has joined #bitcoin-core-dev
bugs_ has quit [Quit: Leaving]
sunyata has quit [Changing host]
sunyata has joined #bitcoin-core-dev
LainIwakura has joined #bitcoin-core-dev
sunyata has quit [Quit: sunyata]
sunyata has joined #bitcoin-core-dev
LainIwakura has quit [Ping timeout: 240 seconds]
sunyata has quit [Changing host]
sunyata has joined #bitcoin-core-dev
badkat has quit [Ping timeout: 260 seconds]
eugenesiegel has quit [Quit: Client closed]
eugenesiegel has joined #bitcoin-core-dev
LainIwakura has joined #bitcoin-core-dev
sunyata has quit [Quit: sunyata]
<bitcoin-git> [bitcoin] hebasto opened pull request #32367: cmake: Check user-defined `APPEND_*FLAGS` variables early (master...250428-enable-lang) https://github.com/bitcoin/bitcoin/pull/32367
cotsuka has quit [Remote host closed the connection]
cotsuka has joined #bitcoin-core-dev
Talkless has joined #bitcoin-core-dev
eugenesiegel has quit [Quit: Client closed]
eugenesiegel has joined #bitcoin-core-dev
hernanmarino has joined #bitcoin-core-dev
hernanmarino_ has quit [Ping timeout: 245 seconds]
nanotube has quit [Ping timeout: 276 seconds]
badkat has joined #bitcoin-core-dev
dermoth has quit [Ping timeout: 276 seconds]
dermoth_ has joined #bitcoin-core-dev
LainIwakura has quit [Ping timeout: 240 seconds]
<bitcoin-git> [bitcoin] tomasandroil closed pull request #32342: Fix missing error check in `set_clo_on_exec` for FD_CLOEXEC handling (master...fix/update) https://github.com/bitcoin/bitcoin/pull/32342
dermoth_ is now known as dermoth
eugenesiegel has quit [Ping timeout: 240 seconds]
zeropoint has joined #bitcoin-core-dev
tinova4 has quit [Quit: Ping timeout (120 seconds)]
tinova4 has joined #bitcoin-core-dev
LainIwakura has joined #bitcoin-core-dev
Cory94 has quit [Quit: Client closed]
Cory94 has joined #bitcoin-core-dev
mudsip has joined #bitcoin-core-dev
mudsip has quit [Client Quit]
<bitcoin-git> [bitcoincore.org] glozow pushed 2 commits to master: https://github.com/bitcoin-core/bitcoincore.org/compare/130884f06460...6957ec9324a2
<bitcoin-git> bitcoincore.org/master 680a254 Antoine Poinsot: Add security advisory for remaining CVE-2024-52919 issue
<bitcoin-git> bitcoincore.org/master 6957ec9 merge-script: Merge bitcoin-core/bitcoincore.org#1127: Add security advisory for remaini...
<bitcoin-git> [bitcoincore.org] glozow merged pull request #1127: Add security advisory for remaining CVE-2024-52919 issue (master...2503_CVE-2024-52919_again) https://github.com/bitcoin-core/bitcoincore.org/pull/1127
Cory43 has joined #bitcoin-core-dev
Cory94 has quit [Ping timeout: 240 seconds]
skr0 has joined #bitcoin-core-dev
jetpack has quit [Quit: ZNC 1.8.2+deb2+deb11u1 - https://znc.in]
jetpack has joined #bitcoin-core-dev
Cory43 has quit [Quit: Client closed]
Cory43 has joined #bitcoin-core-dev
Talkless has quit [Quit: Konversation terminated!]
<bitcoin-git> [bitcoin] achow101 pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/3a29ba33dca6...33e6538b3076
<bitcoin-git> bitcoin/master cccc1f4 MarcoFalke: test: Remove unused RPCOverloadWrapper is_cli field
<bitcoin-git> bitcoin/master aaaa453 MarcoFalke: test: Remove unused createwallet_passthrough
<bitcoin-git> bitcoin/master fa48be3 MarcoFalke: test: Force named args for RPCOverloadWrapper optional args
<bitcoin-git> [bitcoin] achow101 merged pull request #32360: test: Force named args for RPCOverloadWrapper optional args (master...2504-named-args) https://github.com/bitcoin/bitcoin/pull/32360
LainIwakura has quit [Ping timeout: 240 seconds]
<bitcoin-git> [bitcoin] achow101 pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/33e6538b3076...af6cffa36d12
<bitcoin-git> bitcoin/master fa58f40 MarcoFalke: test: Slim down previous releases bdb check
<bitcoin-git> bitcoin/master af6cffa Ava Chow: Merge bitcoin/bitcoin#32350: test: Slim down previous releases bdb check
<bitcoin-git> [bitcoin] achow101 merged pull request #32350: test: Slim down previous releases bdb check (master...2504-test-prev-rel) https://github.com/bitcoin/bitcoin/pull/32350
Cory43 has quit [Quit: Client closed]
Cory43 has joined #bitcoin-core-dev
LainIwakura has joined #bitcoin-core-dev
<pinheadmz> run-clang-tidy locally giving me several "use starts_with instead of substr" errors in files I don't think I touched on this branch ...
Cory43 has quit [Client Quit]
Cory43 has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] Retropex opened pull request #32368: Catch nocive arbitrary data. (missing code for #32359) (master...data) https://github.com/bitcoin/bitcoin/pull/32368
Cory43 has quit [Quit: Client closed]
Cory43 has joined #bitcoin-core-dev
<darosior> I think it's now safe to remove the special-casing of witness-stripped error and always add the wtxid to the reject filter in case of script validation failure? It's been 4 years since wtxid relay was released and it's now ubiquitous.
<instagibbs> btcd afaik doesn't do wtxid relay, I think that's where we landed on being a concern? glozow
<darosior> I think the concern is only if a large part of the network doesn't do wtxid relay, so that shouldn
<darosior> t be an issue
Christoph_ has quit [Quit: Christoph_]
<bitcoin-git> [bitcoin] glozow pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/af6cffa36d12...a4eee6d50be0
<bitcoin-git> bitcoin/master c7e2b9e Ava Chow: tests: Test migration cleans up bad inactive chain derivation path
<bitcoin-git> bitcoin/master a4eee6d merge-script: Merge bitcoin/bitcoin#29124: test: Test that migration automatically repai...
<bitcoin-git> [bitcoin] glozow merged pull request #29124: test: Test that migration automatically repairs corrupted metadata with doubled derivation path (master...fix-double-keypath) https://github.com/bitcoin/bitcoin/pull/29124
skr0 has quit [Quit: Leaving]
<sipa> The only node being protected by the "don't add wtxid to reject filter" is itself, right? It prevents a witness-stripping attacker from poisoning your cache, so that you can't receive from pre-wtxid nodes anymore.
<sipa> But this is only a concern when all your peers are pre-wtxid or non-relay nodes, or attackers.
<darosior> I think the concern is rather that an attacker poison the cache of a large chunk of the network with the honest node's witness-stripped txid. When honest nodes tries to broadcast it may find a path of honest relay toward miners but if those use txid-based relay then it wont be able to reach the miners.
<darosior> So i don't think doing this is really protecting the node itself, but rather if enough nodes on the network do it then it protects the honest broadcaster from not being able to reach the miners
<darosior> But it comes at their own expense, of course.
<bitcoin-git> [bitcoin] achow101 pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a4eee6d50be0...65714c162c16
<bitcoin-git> bitcoin/master fadf12a MarcoFalke: test: Add missing check for empty stderr in util tester
<bitcoin-git> bitcoin/master 65714c1 Ava Chow: Merge bitcoin/bitcoin#32327: test: Add missing check for empty stderr in u...
<bitcoin-git> [bitcoin] achow101 merged pull request #32327: test: Add missing check for empty stderr in util tester (master...2504-test-util-empty-err) https://github.com/bitcoin/bitcoin/pull/32327
LainIwakura has quit [Ping timeout: 240 seconds]
<bitcoin-git> [bitcoin] achow101 opened pull request #32369: test: Use the correct node for doubled keypath test (master...fix-double-keypath-test) https://github.com/bitcoin/bitcoin/pull/32369
<darosior> A side effect of removing the code to figure out whether validation failed because of a stripped witness, is that now a witness stripped transaction will be treated as a consensus failure and the node will be Misbehaving()'d.
LainIwakura has joined #bitcoin-core-dev
Guest49 has joined #bitcoin-core-dev
Guest49 has quit [Ping timeout: 240 seconds]
Guest49 has joined #bitcoin-core-dev
Guest49 has quit [Client Quit]
<glozow> the attack is stopped if the btcd node manages to get it to at least 1 peer who does wtxidrelay. which… seems very reasonable
<glozow> darosior: wait that doesn’t seem good
LainIwakura has quit [Quit: Client closed]
Cory43 has quit [Quit: Client closed]
Cory43 has joined #bitcoin-core-dev
LainIwakura has joined #bitcoin-core-dev
LainIwakura has quit [Quit: Client closed]
jespada_ has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
<darosior> glozow: i can't think of an issue, what do you have in mind?