Bitcoin Core Github
44 subscribers
120K links
Download Telegram
achow101 closed a pull request: "Bugfix: configure: Correct check for fuzz binary needing a main function"
(https://github.com/bitcoin/bitcoin/pull/28564)
achow101 closed a pull request: "Bugfix: Package relay / bytespersigop checks"
(https://github.com/bitcoin/bitcoin/pull/28345)
💬 achow101 commented on pull request "Bugfix: Package relay / bytespersigop checks":
(https://github.com/bitcoin/bitcoin/pull/28345#issuecomment-2045293951)
Closing as up for grabs due to lack of activity.
achow101 closed a pull request: "net: Add new permission `forceinbound` to evict a random unprotected connection if all slots are otherwise full"
(https://github.com/bitcoin/bitcoin/pull/27600)
💬 pinheadmz commented on issue "whiteconnections should be re-added":
(https://github.com/bitcoin/bitcoin/issues/8798#issuecomment-2045309981)
Proposed alternative to #27600 is a setting such that bitcoind listens for inbound connections but does not advertise itself
💬 achow101 commented on pull request "Safegcd-based modular inverses in MuHash3072":
(https://github.com/bitcoin/bitcoin/pull/21590#issuecomment-2045311564)
Are you still working on this?
💬 achow101 commented on pull request "RPC: add new `listmempooltransactions`":
(https://github.com/bitcoin/bitcoin/pull/29016#issuecomment-2045315768)
Are you still working on this?
📝 achow101 converted_to_draft a pull request: "Add OP_TXHASH and OP_CHECKTXHASHVERIFY opcodes"
(https://github.com/bitcoin/bitcoin/pull/29050)
Implementation of OP_TXHASH and OP_CHECKTXHASHVERIFY, as per the [draft BIP](https://github.com/bitcoin/bips/pull/1500).

This MR includes a test using the test vectors generated from the reference implementation in the BIP.
The implementation utilizes a caching strategy that hopefully alleviates concerns around resource usage like quadratic hashing.

This MR on purpose does not make any consensus changes yet. Activation of the proposed opcodes will be coordinated in an independent MR, prob
...
💬 achow101 commented on pull request "Add OP_TXHASH and OP_CHECKTXHASHVERIFY opcodes":
(https://github.com/bitcoin/bitcoin/pull/29050#issuecomment-2045322740)
Seems like this should be a draft
📝 achow101 converted_to_draft a pull request: "indexes: Stop using node internal types and locking cs_main, improve sync logic"
(https://github.com/bitcoin/bitcoin/pull/24230)
This PR lets indexing code mostly run outside of the node process. It also improves indexing sync logic, which is moved out of indexing code to a new [`node::SyncChain()`](https://github.com/ryanofsky/bitcoin/blob/8d8cdcb37005030f646ba3c45f7f54f556efb8bf/src/node/chain.h#L24-L39) function.

Almost all the commits in this PR are small refactoring changes that move code from `src/index/` to `src/node/`, or replace references to node types like `CBlockIndex`, `CChain`, `CChainState` in index code
...
achow101 closed a pull request: "contrib/init: (OpenRC) use -daemonwait to wait for startup completion"
(https://github.com/bitcoin/bitcoin/pull/24066)
💬 achow101 commented on pull request "contrib/init: (OpenRC) use -daemonwait to wait for startup completion":
(https://github.com/bitcoin/bitcoin/pull/24066#issuecomment-2045334717)
> Are we just going to concede that the problem is unsolvable?

This PR does not seem to have conceptual support.
achow101 closed a pull request: "net: call getaddrinfo() in detachable thread to prevent stalling"
(https://github.com/bitcoin/bitcoin/pull/27557)
achow101 closed a pull request: "wallet: add `seeds` argument to `importdescriptors`"
(https://github.com/bitcoin/bitcoin/pull/27351)
💬 achow101 commented on pull request "wallet: add `seeds` argument to `importdescriptors`":
(https://github.com/bitcoin/bitcoin/pull/27351#issuecomment-2045348329)
The approach in this PR does not seem to have support.
⚠️ Zhouziheng3188 opened an issue: "Can't version 26.1 export Bech32 address private key?"
(https://github.com/bitcoin/bitcoin/issues/29836)
bitcoin core -v 26.1.0

Dumpprivkey instruction can't export the private key. What is the reason?

![image](https://github.com/bitcoin/bitcoin/assets/38863947/0a6387f2-198a-4743-aeac-e05ec8e7a887)
💬 maflcko commented on issue "Can't version 26.1 export Bech32 address private key?":
(https://github.com/bitcoin/bitcoin/issues/29836#issuecomment-2045361385)
Only legacy wallets are supported by this command. Does `listdescriptors` work for you?
💬 achow101 commented on pull request "Introduce 'getblockfileinfo' RPC command":
(https://github.com/bitcoin/bitcoin/pull/27770#issuecomment-2045362077)
Are you still working on this?
💬 laanwj commented on pull request "security: restrict abis in bitcoind.service":
(https://github.com/bitcoin/bitcoin/pull/28340#issuecomment-2045362990)
ACK 0244416aacbad03e4ebe8f2c95c7861a318916ea . This is a sensible security feature.
It looks like [the documentation](https://www.freedesktop.org/software/systemd/man/latest/systemd.exec.html) of systemd.exec even mentions this pairing.

I do agree with @sjors, however, that when we're adding systemd hardening options, we might as well look further.
achow101 closed a pull request: "Fix issues in ZMQ error handling"
(https://github.com/bitcoin/bitcoin/pull/28029)