ChanServ 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 @ 14:00 UTC | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
johnny9dev58450 has quit [Ping timeout: 260 seconds]
johnny9dev58450 has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
johnny9dev58450 has quit [Ping timeout: 272 seconds]
brunoerg has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
jonatack has quit [Ping timeout: 272 seconds]
jonatack has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
<bitcoin-git> [guix.sigs] pinheadmz opened pull request #1088: pinheadmz attestation 26.1rc2 non-codesigned (main...26.1rc2) https://github.com/bitcoin-core/guix.sigs/pull/1088
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
johnny9dev58450 has joined #bitcoin-core-dev
test_ has joined #bitcoin-core-dev
flooded has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
realies has quit [Read error: Connection reset by peer]
realies has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 272 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 240 seconds]
jarthur has quit [Quit: jarthur]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
no-world has joined #bitcoin-core-dev
<no-world> Hi!  When will Bitcoin read legacy wallets addresses created with 2009-2013 Bitcoin core hardware wallets?  And, when will the coin generation feature be added back?  And, under the MIT license, can I or the Nevada Gaming Commission do these things and more?  I believe I know, but I prefer to ask again!
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
no-world has quit [Quit: Client closed]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 255 seconds]
puchka has quit [Ping timeout: 255 seconds]
puchka has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
Chris_Stewart_5 has quit [Ping timeout: 246 seconds]
puchka has quit [Ping timeout: 252 seconds]
BrandonOdiwuor has joined #bitcoin-core-dev
Chris_Stewart_5 has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
dev7ba has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
<bitcoin-git> [guix.sigs] achow101 merged pull request #1088: pinheadmz attestation 26.1rc2 non-codesigned (main...26.1rc2) https://github.com/bitcoin-core/guix.sigs/pull/1088
<bitcoin-git> [guix.sigs] achow101 pushed 2 commits to main: https://github.com/bitcoin-core/guix.sigs/compare/a34b563aa5eb...1d3ebaba3f35
<bitcoin-git> guix.sigs/main 999672f Matthew Zipkin: pinheadmz attestation 26.1rc2 non-codesigned
<bitcoin-git> guix.sigs/main 1d3ebab Ava Chow: Merge pull request #1088 from pinheadmz/26.1rc2
BrandonOdiwuor has quit [Quit: Client closed]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
Srishti has joined #bitcoin-core-dev
Srishti has quit [Quit: Client closed]
brunoerg has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
BrandonOdiwuor has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a945f09fa6e0...e70590988a7b
<bitcoin-git> bitcoin/master 8aff3fd fanquake: depends: don't use -h with touch on OpenBSD
<bitcoin-git> bitcoin/master e705909 fanquake: Merge bitcoin/bitcoin#29598: depends: don't use -h with touch on OpenBSD
<bitcoin-git> [bitcoin] fanquake merged pull request #29598: depends: don't use -h with touch on OpenBSD (master...open_touch_no_h) https://github.com/bitcoin/bitcoin/pull/29598
agrosant has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/e70590988a7b...31be1a47675e
<bitcoin-git> bitcoin/master fa9a5e8 MarcoFalke: refactor: Add missing {} around error() calls
<bitcoin-git> bitcoin/master fa1d624 MarcoFalke: scripted-diff: return error(...); ==> error(...); return false;
<bitcoin-git> bitcoin/master fa808fb MarcoFalke: refactor: Make error() return type void
<bitcoin-git> [bitcoin] fanquake merged pull request #29236: log: Nuke error(...) (master...2401-log-error-) https://github.com/bitcoin/bitcoin/pull/29236
kevkevin has quit [Remote host closed the connection]
Guyver2 has left #bitcoin-core-dev [Closing Window]
BrandonOdiwuor has quit [Quit: Client closed]
BrandonOdiwuor has joined #bitcoin-core-dev
agrosant has quit [Ping timeout: 264 seconds]
<bitcoin-git> [bitcoin-maintainer-tools] achow101 pushed 2 commits to main: https://github.com/bitcoin-core/bitcoin-maintainer-tools/compare/c93dcfec0b97...9e350dc74b2e
<bitcoin-git> bitcoin-maintainer-tools/main 9333d5f Jon Atack: github-merge: omit acks in quoted comments or markdown indented
<bitcoin-git> bitcoin-maintainer-tools/main 9e350dc Ava Chow: Merge bitcoin-core/bitcoin-maintainer-tools#154: github-merge: omit ACKs i...
<bitcoin-git> [bitcoin-maintainer-tools] achow101 merged pull request #154: github-merge: omit ACKs in quoted comments or with markdown indentation (main...2023-06-omit-acks-in-quoted-comments-and-markdown-indentation) https://github.com/bitcoin-core/bitcoin-maintainer-tools/pull/154
abubakarsadiq has joined #bitcoin-core-dev
agrosant has joined #bitcoin-core-dev
BrandonOdiwuor has quit [Quit: Client closed]
Chris_Stewart_5 has quit [Ping timeout: 260 seconds]
Chris_Stewart_5 has joined #bitcoin-core-dev
test_ is now known as _flood
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/31be1a47675e...bd55b7a528df
<bitcoin-git> bitcoin/master ae5f720 Hennadii Stepanov: ci: Add workaround for Homebrew's python link error
<bitcoin-git> bitcoin/master acc06bc Hennadii Stepanov: ci, macos: Use `--break-system-packages` with Homebrew's python
<bitcoin-git> bitcoin/master bd55b7a fanquake: Merge bitcoin/bitcoin#29610: ci: Fix "macOS native" job
<bitcoin-git> [bitcoin] fanquake merged pull request #29610: ci: Fix "macOS native" job (master...240309-homebrew) https://github.com/bitcoin/bitcoin/pull/29610
Chris_Stewart_5 has quit [Ping timeout: 268 seconds]
Chris_Stewart_5 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/bd55b7a528df...d14c7286b653
<bitcoin-git> bitcoin/master 115c283 Max Edwards: ci: add print of powershell version to win64 job
<bitcoin-git> bitcoin/master d14c728 fanquake: Merge bitcoin/bitcoin#29620: ci: add print of powershell version to win64 ...
<bitcoin-git> [bitcoin] fanquake merged pull request #29620: ci: add print of powershell version to win64 job (master...psversion) https://github.com/bitcoin/bitcoin/pull/29620
BrandonOdiwuor has joined #bitcoin-core-dev
brunoerg has quit [Remote host closed the connection]
BrandonOdiwuor has quit [Quit: Client closed]
agrosant has quit [Ping timeout: 264 seconds]
BrandonOdiwuor has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fjahr opened pull request #29633: log: Remove error() reference (master...2024-03-error-cleanup) https://github.com/bitcoin/bitcoin/pull/29633
agrosant has joined #bitcoin-core-dev
agrosant has quit [Ping timeout: 264 seconds]
jespada_ has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 256 seconds]
kevkevin has joined #bitcoin-core-dev
agrosant has joined #bitcoin-core-dev
agrosant has quit [Ping timeout: 264 seconds]
agrosant has joined #bitcoin-core-dev
agrosant has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
bugs_ has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 264 seconds]
jespada_ has quit [Ping timeout: 252 seconds]
preimage has joined #bitcoin-core-dev
jespada has joined #bitcoin-core-dev
justache is now known as fotastache
brunoerg has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] 0xB10C opened pull request #29636: test: #29007 follow ups (master...2024-03-29007-follow-ups) https://github.com/bitcoin/bitcoin/pull/29636
<bitcoin-git> [bitcoin] achow101 closed pull request #29530: rpc/net: Adds misbehaving_score to getpeerinfo (master...2024-03-01-getpeerinfo-misbehaving) https://github.com/bitcoin/bitcoin/pull/29530
<vasild> What is the point of using SaltedTxidHasher for unordered_map/set of tx ids? Why not just use e.g. the first 64 bits of the tx id? What's the point of using a salt? Surely we are not worred that a malicious actor could send us transactions with special ids to ruin the quality of our unordered_set containers and turn them into O(n) search/insert/delete containers.
<vasild> ^ related to pinheadmz's question at https://github.com/bitcoin/bitcoin/pull/29415#discussion_r1518225842
<bitcoin-git> [bitcoin] achow101 pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/d14c7286b653...12dae637a472
<bitcoin-git> bitcoin/master b5d15f7 glozow: [refactor] return pair from SingleV3Checks
<bitcoin-git> bitcoin/master 1703067 glozow: [policy] sibling eviction for v3 transactions
<bitcoin-git> bitcoin/master 5fbab37 glozow: [unit test] sibling not returned from SingleV3Checks if 1p2c or 3gen
<bitcoin-git> [bitcoin] achow101 merged pull request #29306: policy: enable sibling eviction for v3 transactions (master...2024-01-sibling-eviction) https://github.com/bitcoin/bitcoin/pull/29306
AaronvanW has joined #bitcoin-core-dev
Guest82 has joined #bitcoin-core-dev
Guest82 has quit [Client Quit]
jarthur has joined #bitcoin-core-dev
<sipa> vasild: that is exactly what we're worried about
<vasild> L-O
<vasild> that would require work by the malicious actor, right? generating transactions with pre-determined ids
<sipa> it wouldn't need much grinding
<vasild> and to cause troubles they have to generate 1000s of them
<sipa> yes
<sipa> hashing is pretty cheap, bitcoin may have encourage some hardware developments in that regard
<vasild> ok, switching my container to use saltedhasher then...
<instagibbs> sipa citation needed
<vasild> hmm, for the private broadcast where we store only locally submitted transactions in the unordered_map it is less of a concern... will use SaltedTxidHasher anyway
<sipa> vasild: yeah, i think it's just good best practice; there are probably some data structures where it doesn't actually matter
<sipa> but it's easier not having to reason about that
<vasild> right
<sipa> instagibbs: SHA256(SHA256(0000402054918d671610fc65c9acb5ddb46d30e1c25194daa00d050000000000000000005a0058b0c79451ff7883a5804af798ba2c55bb62533e46b7396edffa1e6fc4622d8c3b63aef908178c0f23c1)) = 0000000000000000000000005d6f06154c8685146aa7bc3dc9843876c9cefd0f (all hex)
<vasild> maybe even introduce something like using TxidMap = std::unordered_map<Txid, V, SaltedTxidHasher>
<sipa> instagibbs: with 97 zero bits up front
<vasild> (and use it all over the place)
<instagibbs> wow, that's a nice block
<bitcoin-git> [bitcoin] achow101 pushed 8 commits to master: https://github.com/bitcoin/bitcoin/compare/12dae637a472...bef99176e638
<bitcoin-git> bitcoin/master 2863d7d brunoerg: net: store `-whitelist{force}relay` values in `CConnman`
<bitcoin-git> bitcoin/master 9133fd6 Luke Dashjr: net: Move `NetPermissionFlags::Implicit` verification to `AddWhitelistPerm...
<bitcoin-git> bitcoin/master 8e06be3 Luke Dashjr: net_processing: Move extra service flag into InitializeNode
<bitcoin-git> [bitcoin] achow101 merged pull request #27114: p2p: Allow whitelisting manual connections (master...2023-02-outgoing-whitelist) https://github.com/bitcoin/bitcoin/pull/27114
<pinheadmz> sipa not a very nice transaction though ?
<sipa> pinheadmz: hmm?
<pinheadmz> 0000402054918d671610fc65c9acb5ddb46d30e1c25194daa00d050000000000000000005a0058b0c79451ff7883a5804af798ba2c55bb62533e46b7396edffa1e6fc4622d8c3b63aef908178c0f23c1
<pinheadmz> was that supposed to be a brute forced tx with hash collision?
<sipa> no, it's block header 756951
<sipa> as proof that SHA256 is fairly easy nowadays
<pinheadmz> check +1
<bitcoin-git> [bitcoin] achow101 pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/bef99176e638...bde3db40f6d5
<bitcoin-git> bitcoin/master da338aa Andrew Toth: blockstorage: check nPos in ReadRawBlockFromDisk before seeking back
<bitcoin-git> bitcoin/master 38265cc Andrew Toth: zmq: read raw block with ReadRawBlockFromDisk
<bitcoin-git> bitcoin/master 0865ab8 Andrew Toth: test: check more details on zmq raw block response
<bitcoin-git> [bitcoin] achow101 merged pull request #26415: rpc,rest,zmq: faster getblock, NotifyBlock and rest_block by reading raw block (master...read-raw-block) https://github.com/bitcoin/bitcoin/pull/26415
kevkevin has quit [Remote host closed the connection]
fotastache has quit [Quit: ZNC 1.8.2 - https://znc.in]
dzxzg has joined #bitcoin-core-dev
<dzxzg> #proposedmeetingtopic Topics for the 27.0 Testing Guide
Talkless has joined #bitcoin-core-dev
brunoerg has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] mzumsande opened pull request #29639: test: fix intermittent failures with test=addrman (master...202403_fix_feature_asmap) https://github.com/bitcoin/bitcoin/pull/29639
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
Randolf has joined #bitcoin-core-dev
oneeyedalien has joined #bitcoin-core-dev
vasild has quit [Ping timeout: 260 seconds]
BrandonOdiwuor has quit [Quit: Client closed]
vasild has joined #bitcoin-core-dev
dev7ba has quit [Quit: Client closed]
oneeyedalien has quit [Quit: Leaving]
<bitcoin-git> [guix.sigs] hebasto opened pull request #1089: Add attestations by hebasto for 26.1rc2 non-codesigned (main...26.1rc2-non-codesigned) https://github.com/bitcoin-core/guix.sigs/pull/1089
dzxzg has quit [Quit: Client closed]
<bitcoin-git> [guix.sigs] fanquake merged pull request #1089: Add attestations by hebasto for 26.1rc2 non-codesigned (main...26.1rc2-non-codesigned) https://github.com/bitcoin-core/guix.sigs/pull/1089
<bitcoin-git> [guix.sigs] fanquake pushed 2 commits to main: https://github.com/bitcoin-core/guix.sigs/compare/1d3ebaba3f35...79780281e041
<bitcoin-git> guix.sigs/main 309ee73 Hennadii Stepanov: Add attestations by hebasto for 26.1rc2 non-codesigned
<bitcoin-git> guix.sigs/main 7978028 fanquake: Merge pull request #1089 from hebasto/26.1rc2-non-codesigned
dzxzg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/bde3db40f6d5...1105aa46dd10
<bitcoin-git> bitcoin/master d0e6564 Fabian Jahr: log: Remove error() reference
<bitcoin-git> bitcoin/master 1105aa4 fanquake: Merge bitcoin/bitcoin#29633: log: Remove error() reference
<bitcoin-git> [bitcoin] fanquake merged pull request #29633: log: Remove error() reference (master...2024-03-error-cleanup) https://github.com/bitcoin/bitcoin/pull/29633
Randolf has quit [Quit: Leaving]
dzxzg has quit [Quit: Client closed]
dzxzg has joined #bitcoin-core-dev
dzxzg has quit [Client Quit]
dzxzg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] sipa closed pull request #29284: Choose earliest-activatable as tie breaker between equal-work chains (master...202401_better_block_tiebreak) https://github.com/bitcoin/bitcoin/pull/29284
Talkless has quit [Ping timeout: 272 seconds]
dzxzg has quit [Ping timeout: 240 seconds]
<bitcoin-git> [bitcoin] sr-gi opened pull request #29640: Adds missing test to chain ties (CBlockIndexWorkComparator) (master...202403-block-tiebreak) https://github.com/bitcoin/bitcoin/pull/29640
<bitcoin-git> [bitcoin] maflcko opened pull request #29641: scripted-diff: Use LogInfo/LogDebug over LogPrintf/LogPrint (master...2403-log-) https://github.com/bitcoin/bitcoin/pull/29641
dzxzg has joined #bitcoin-core-dev
bugs_ has quit [Quit: Leaving]
jarthur has quit [Ping timeout: 268 seconds]
Akshat19 has joined #bitcoin-core-dev
jarthur has joined #bitcoin-core-dev
preimage has quit [Quit: WeeChat 4.2.1]
szkl has quit [Quit: Connection closed for inactivity]
Akshat19 has quit [Ping timeout: 250 seconds]
<bitcoin-git> [bitcoin] TheCharlatan opened pull request #29642: kernel: Handle fatal errors through return values (master...returnTypeShutdown) https://github.com/bitcoin/bitcoin/pull/29642
dzxzg has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin-detached-sigs] achow101 pushed 1 commit to 26.x: https://github.com/bitcoin-core/bitcoin-detached-sigs/compare/8576b8b29622...7139b57ea6e1
<bitcoin-git> bitcoin-detached-sigs/26.x 7139b57 Matthew Zipkin: 26.1: macOS signature for rc2
dzxzg has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
flooded has joined #bitcoin-core-dev
_flood has quit [Ping timeout: 255 seconds]
SpellChecker_ has joined #bitcoin-core-dev
SpellChecker has quit [Ping timeout: 260 seconds]
brunoerg_ has joined #bitcoin-core-dev
brunoerg_ has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev