<laanwj>
davidbakin: if you don't get an answer here, you might want to ask in #bitcoin-wizards
2022-06-01
<bitcoin-git>
[bitcoin] theuni opened pull request #25264: kernel: pass params to BlockManager rather than using a global (master...no-kernel-global-params) https://github.com/bitcoin/bitcoin/pull/25264
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25262: test: Set maxfeerate=0 in MiniWallet sendrawtransaction() (master...2206-test-mini-fast-📨) https://github.com/bitcoin/bitcoin/pull/25262
<bitcoin-git>
[bitcoin] fanquake opened pull request #25263: build: pass minisketch cppflags only where needed (master...minisketch_cpp_needed) https://github.com/bitcoin/bitcoin/pull/25263
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25262: test: Set maxfeerate=0 in MiniWallet sendrawtransaction() (master...2206-test-mini-fast-📨) https://github.com/bitcoin/bitcoin/pull/25262
<bitcoin-git>
[bitcoin] fanquake merged pull request #25244: build: pass bdb cppflags only where needed (master...use_bdb_cppflags_needed) https://github.com/bitcoin/bitcoin/pull/25244
<bitcoin-git>
bitcoin/master 46a8909 fanquake: build: pass bdb cppflags only where needed
<bitcoin-git>
bitcoin/master b752dad fanquake: Merge bitcoin/bitcoin#25244: build: pass bdb cppflags only where needed
<bitcoin-git>
[bitcoin] MarcoFalke closed pull request #25257: p2p: Disconnect manual peers and not discourage all peers with the same address on getblocktxn msg corruption (master...2205-no-misbehaving-🌷) https://github.com/bitcoin/bitcoin/pull/25257
<bitcoin-git>
[bitcoin] ryanofsky reopened pull request #22764: build: Include qt sources for parsing with extract_strings.py (master...210821-translation) https://github.com/bitcoin/bitcoin/pull/22764
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25257: p2p: Disconnect manual peers and not discourage all peers with the same address on getblocktxn msg corruption (master...2205-no-misbehaving-🌷) https://github.com/bitcoin/bitcoin/pull/25257
<bitcoin-git>
[bitcoin] klementtan opened pull request #25256: logging: Add leveldb thread name and fix logging empty thread name (master...add_leveldb_threadname) https://github.com/bitcoin/bitcoin/pull/25256
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #24178: p2p: Respond to getheaders if we have sufficient chainwork (master...2022-01-headers-response-requires-minchainwork) https://github.com/bitcoin/bitcoin/pull/24178
<bitcoin-git>
bitcoin/master a35f963 Suhas Daftuar: Add test for getheaders behavior
<bitcoin-git>
bitcoin/master 5f65aff MacroFake: Merge bitcoin/bitcoin#24178: p2p: Respond to getheaders if we have suffici...
<bitcoin-git>
[bitcoin] brunoerg opened pull request #25253: test: add coverage for non-hex value to -minimumchainwork (master...2022-05-minimumchainwork-nonhex) https://github.com/bitcoin/bitcoin/pull/25253
<bitcoin-git>
[gui] hebasto opened pull request #612: refactor: Drop unused `QFrame`s in `SendCoinsEntry` (master...220530-sendcoins) https://github.com/bitcoin-core/gui/pull/612
<theStack>
sipa: thanks, good to know. that explains why there are PRs around implementing elligator squared pubkeys (both in secp256k1 and bitcoin core), but no mention of it in the (obviously outdated) spec i read
<theStack>
bip324 question about handshake: in older slides (from breaking bitcoin 2019) i saw it was proposed that only odd pubkeys (i.e. starting with 0x03) are allowed, but the current BIP states only even pubkeys (i.e. starting with 0x02) are. any specific reason for this change?
<sipa>
And Bitcoin Core from back then cannot be used unmodified against today's OpenSSL's versions.
<sipa>
Well, it didn't happen with Bitcoin Core versions from 2014 linked against OpenSSL from 2014 and before.
<sipa>
but i believe that no version of bitcoin core (including very old ones) would emit such encodings
<sipa>
there are certainly DER private key encodings that are valid but bitcoin core today wouldn't be able to parse, as it doesn't have a full DER parser
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25250: [22.x] test: replace hashlib.ripemd160 with an own implementation (22.x...2205-backport-ripemd160-👥) https://github.com/bitcoin/bitcoin/pull/25250
<sipa>
it's not clear to me whether this is an issue for the bitcoin core codebase at all (includin historical versions)
<DavidBakin>
so that means someone with such a wallet can't in fact use a newer bitcoin core to read it?
<div72>
laanwj: You're right, I've encountered the issue in a downstream where OpenSSL was still being used. The chances of this affecting Bitcoin is pretty slim as a wallet from a 7y old version which needs to be compiled with a pre 3 recent OpenSSL version is needed.
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25044: test: Use MiniWallet in rpc_rawtransaction.py (master...test_rawtransaction_miniwallet) https://github.com/bitcoin/bitcoin/pull/25044
<bitcoin-git>
bitcoin/master 269fa66 MacroFake: Merge bitcoin/bitcoin#25044: test: Use MiniWallet in rpc_rawtransaction.py
<bitcoin-git>
[gui] hebasto opened pull request #611: test, refactor: Use the same shutdown path as in bitcoin-qt binary (master...220530-exec2) https://github.com/bitcoin-core/gui/pull/611
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25242: [22.x] rpc: Capture potentially large UniValue by ref for rpcdoccheck (22.x...2205-uni-no-copy-📊) https://github.com/bitcoin/bitcoin/pull/25242
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25241: [23.x] rpc: Capture potentially large UniValue by ref for rpcdoccheck (23.x...2205-uni-no-copy-📊) https://github.com/bitcoin/bitcoin/pull/25241
<bitcoin-git>
bitcoin/master 20ff499 Martin Zumsande: rpc: Capture potentially large UniValue by ref for rpcdoccheck
<laanwj>
div72: how do you even get a foreign key into that function? i don't think it's considered an issue because afaik, that function is only used for loading wallets, and it only supports the keys generated by bitcoin core itself
<bitcoin-git>
bitcoin/master 387ae8b Sebastian Falbesoner: rpc: remove deprecated fee fields from mempool entries
<bitcoin-git>
[bitcoin] 1440000bytes closed pull request #25236: wallet: use vector instead of list for transactions (master...listtx-memory) https://github.com/bitcoin/bitcoin/pull/25236
2022-05-29
<bitcoin-git>
[bitcoin] fanquake merged pull request #25231: ci: Install documented packages for "Win64" CI task (master...220528-posix) https://github.com/bitcoin/bitcoin/pull/25231
<bitcoin-git>
[bitcoin] furszy opened pull request #25239: wallet: 'CommitTransaction', remove extra wtx lookup and add exception for db write error (master...2022_wallet_CommitTransaction_extra_lookup) https://github.com/bitcoin/bitcoin/pull/25239
<bitcoin-git>
[bitcoin] kristapsk opened pull request #25238: script: Disable SC1004 check in `contrib/devtools/gen-bitcoin-conf.sh` (master...shellcheck-disable=SC1004) https://github.com/bitcoin/bitcoin/pull/25238
<bitcoin-git>
[gui] hebasto opened pull request #609: qt, wallet, refactor: Drop unused `WalletModel::PaymentRequestExpired` (master...220529-expired) https://github.com/bitcoin-core/gui/pull/609
<bitcoin-git>
[gui] hebasto opened pull request #608: qt, wallet, refactor: Make `WalletModel::sendCoins()` return `void` (master...220529-sendcoins) https://github.com/bitcoin-core/gui/pull/608
<bitcoin-git>
[bitcoin] mzumsande opened pull request #25237: rpc: Capture UniValue by ref for rpcdoccheck (master...202205_rpcdoc_capture) https://github.com/bitcoin/bitcoin/pull/25237
2022-05-28
<bitcoin-git>
[bitcoin] 1440000bytes opened pull request #25236: wallet: use vector instead of list for transactions (master...listtx-memory) https://github.com/bitcoin/bitcoin/pull/25236
<bitcoin-git>
[bitcoin] amadeuszpawlik opened pull request #25235: GetExternalSigner(): fail if multiple signers are found (master...fail_with_multiple_ext_signers) https://github.com/bitcoin/bitcoin/pull/25235
<bitcoin-git>
[bitcoin] furszy opened pull request #25234: bench: add benchmark for wallet 'AvailableCoins' function. (master...2022_bench_wallet_available_coins) https://github.com/bitcoin/bitcoin/pull/25234
<bitcoin-git>
[bitcoin] hebasto opened pull request #25231: ci: Install documented packages for "Win64" CI task (master...220528-posix) https://github.com/bitcoin/bitcoin/pull/25231
<bitcoin-git>
[bitcoin] hebasto reopened pull request #20728: ci: Fix COMMIT_RANGE variable value for cloned repos (master...201220-cirrus) https://github.com/bitcoin/bitcoin/pull/20728
<bitcoin-git>
[bitcoin] fanquake merged pull request #25224: Get time less often in AddrManImpl::ResolveCollisions_() (master...2205-time-less-🐠) https://github.com/bitcoin/bitcoin/pull/25224
<DavidBakin>
it does, it's just a fork of bitcoin/bitcoin - oh wait, i read something in the cirrus ci docs that it _only pulls your branch_ that triggered it - let me investigate
2022-05-27
<bitcoin-git>
[bitcoin] jamesob opened pull request #25228: test: add BIP-125 rule 5 testcase with default mempool (master...2022-05-bip125-rule5-test) https://github.com/bitcoin/bitcoin/pull/25228
<DavidBakin>
huh. i just want to be able to, for example, pass lint and win64 on my own rather than wait for bitcoin/bitcoin to find it in my PR. thanks!
<_aj_>
omg, setting up CI for a bitcoin fork is easy now? <3 you guys
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25227: Return empty vector on invalid hex encoding (master...2205-reject-non-hex-🌲) https://github.com/bitcoin/bitcoin/pull/25227
<bitcoin-git>
[bitcoin] fanquake merged pull request #25214: multiprocess build fix: ipc/capnp/init.capnp.h: No such file or directory (master...pr/mpdep) https://github.com/bitcoin/bitcoin/pull/25214
<bitcoin-git>
bitcoin/master 44904aa Ryan Ofsky: multiprocess build cleanup: comment on manual dependencies
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25224: Get time less often in AddrManImpl::ResolveCollisions_() (master...2205-time-less-🐠) https://github.com/bitcoin/bitcoin/pull/25224
<bitcoin-git>
[bitcoin] brunoerg opened pull request #25220: rpc: fix incorrect warning for address type p2sh-segwit in createmultisig (master...2022-05-fix-incorrect-warning-createmultisig) https://github.com/bitcoin/bitcoin/pull/25220
<bitcoin-git>
bitcoin/master 192d639 Andrew Chow: Merge bitcoin/bitcoin#25213: fuzz: fix crash at coinselection, add missing...
<bitcoin-git>
[bitcoin] natanleung opened pull request #25219: test: deploymentinfo.cpp unit tests (master...deploymentinfo_unit_test) https://github.com/bitcoin/bitcoin/pull/25219
<bitcoin-git>
[bitcoin] furszy opened pull request #25218: wallet: introduce generic 'Result' classes and connect it to `CreateTransaction` and `GetNewDestination` (master...2022_generic_result) https://github.com/bitcoin/bitcoin/pull/25218
<bitcoin-git>
[bitcoin] jonatack opened pull request #25217: scripts and tools: update lint-logs.py with LogPrintLevel() (master...update-logs-linter-script-with-LogPrintLevel) https://github.com/bitcoin/bitcoin/pull/25217
<bitcoin-git>
[bitcoin] ryanofsky opened pull request #25214: libmultiprocess build fix: ipc/capnp/init.capnp.h: No such file or directory (master...pr/mpdep) https://github.com/bitcoin/bitcoin/pull/25214
<bitcoin-git>
[bitcoin] furszy opened pull request #25213: fuzz: fix crash at coinselection, add missing fee rate. (master...2022_fuzz_solve_coinselection) https://github.com/bitcoin/bitcoin/pull/25213
<sipa>
TallTim: This channel is about the development of the Bitcoin Core software, an implementation of the Bitcoin protocol. It is not the place to discuss the direction the Bitcoin protocol itself is going.
<bitcoin-git>
[bitcoin] laanwj merged pull request #25197: contrib: Remove keys that are no longer used for merging (master...220524-keys) https://github.com/bitcoin/bitcoin/pull/25197
<bitcoin-git>
bitcoin/master d4b3dc5 Hennadii Stepanov: contrib: Remove keys that are no longer used for merging
<bitcoin-git>
bitcoin/master b4f6869 laanwj: Merge bitcoin/bitcoin#25197: contrib: Remove keys that are no longer used ...
<bitcoin-git>
[bitcoin] laanwj merged pull request #25180: [22.x] qt: Avoid crash on startup if int specified in settings.json (22.x...pr/setint22) https://github.com/bitcoin/bitcoin/pull/25180
<bitcoin-git>
bitcoin/22.x 344537c Ryan Ofsky: qt: Avoid crash on startup if int specified in settings.json
<bitcoin-git>
bitcoin/22.x 04fdd64 laanwj: Merge bitcoin/bitcoin#25180: [22.x] qt: Avoid crash on startup if int spec...