✅ maflcko closed an issue: "A missing import to the src/chainparamsbase.h"
(https://github.com/bitcoin/bitcoin/issues/33019)

💬 maflcko commented on issue "A missing import to the src/chainparamsbase.h":
(https://github.com/bitcoin/bitcoin/issues/33019#issuecomment-3095495511)
Fixed in 138f8671569f7ebb8c84e9d80c44cddeda9e3845

💬 maflcko commented on issue "A missing import to the src/chainparamsbase.h":
(https://github.com/bitcoin/bitcoin/issues/33019#issuecomment-3095504166)
@totdking As this was fixed in all supported versions of bitcoin core, you are likely running an EOL version. It could make sense to consider updating it. See https://bitcoincore.org/en/lifecycle/

✅ maflcko closed an issue: "IP leak using bitcoin core"
(https://github.com/bitcoin/bitcoin/issues/33027)

💬 maflcko commented on issue "IP leak using bitcoin core":
(https://github.com/bitcoin/bitcoin/issues/33027#issuecomment-3095524743)
Not sure what your question is, but Bitcoin Core connects to other P2P nodes. If you to connect to them over tor, please refer to the existing docs, such as https://github.com/bitcoin/bitcoin/blob/master/doc/tor.md, or the config help.


Usually the issue tracker is used to track technical issues related to the Bitcoin Core code base.

General bitcoin questions and/or support requests are best directed to the [Bitcoin StackExchange](https://bitcoin.stackexchange.com) or the `#bitcoin` IRC channe
...

💬 Sjors commented on issue "intermittent timeout in wallet_signer.py : sendall timed out":
(https://github.com/bitcoin/bitcoin/issues/33015#issuecomment-3095573346)
Same underlying issue as https://github.com/bitcoin/bitcoin/issues/32855#issuecomment-3092571841? Both `send` and `sendall` use the external signer method to send the PSBT over to the device (mock).

The additional logging from #32928 may be helpful to narrow down where the stalling happens.

💬 maflcko commented on pull request "tests: speed up coins_tests by parallelizing":
(https://github.com/bitcoin/bitcoin/pull/32945#issuecomment-3095605427)
lgtm ACK 2a8fdddd4df4b630c0a580f4df6521cb3af01804

💬 maflcko commented on pull request "tests: speed up coins_tests by parallelizing":
(https://github.com/bitcoin/bitcoin/pull/32945#discussion_r2218430945)
```suggestion
'`foo_tests`, or if there are multiple test suites, `foo_tests_bar`.\n'
```

style nit: seems odd to use multiple format styles in the same line.

💬 maflcko commented on pull request "clang-format: align brace-after-struct and *-class formatting":
(https://github.com/bitcoin/bitcoin/pull/32813#discussion_r2218452297)
unrelated note: I understand the setting `IncludeBlocks` is set to `Preserve` (and it should probably stay this way by default, but it would be nice to update the other Include* settings (in another pull request) to properly sort the includes: (1) the main include, (2) everything else (Bitcoin Core includes) (3) third-party library includes (boost, Qt), (4) stdlib includes (those that do not end in `.h`). This way, one could toggle the `IncludeBlocks` to get the desired behavior.

✅ maflcko closed a pull request: "refactor: simplify GetAncestor"
(https://github.com/bitcoin/bitcoin/pull/31778)

💬 maflcko commented on pull request "refactor: simplify GetAncestor":
(https://github.com/bitcoin/bitcoin/pull/31778#issuecomment-3095681699)
Closing for now due to inactivity. Feel free to ask for this to be re-opened or you may open a fresh pull request, once you are working on this again.

💬 Sjors commented on issue "wallet: render BIP388 wallet policies":
(https://github.com/bitcoin/bitcoin/issues/32659#issuecomment-3095726908)
Meanwhile I started working on #33008 which has most of the functionality to send a BIP388 policy to an external signer, get a resulting hmac and store that.

But I haven't worked on converting our descriptors to BIP388 policies yet.

🤔 Sjors reviewed a pull request: "test: add test cases to wallet_signer.py"
(https://github.com/bitcoin/bitcoin/pull/33020#pullrequestreview-3037095566)
These tests have been disabled from the start in d4b0107d68a91ed4d1a5c78c8ca76251329d3f3c of #16546.

For the third commit 74fb47f071451145b550062e382416db9388433d: I suggest deleting the commented out code. It's not doing anything useful for the test, and I can't remember why I wrote it.

💬 Sjors commented on pull request "test: add test cases to wallet_signer.py":
(https://github.com/bitcoin/bitcoin/pull/33020#discussion_r2218550645)
38a6293fa89104f3d6f1907d04c2e4c488157307: this makes it more clear how `set_mock_result` is intended to be used:

```py
self.set_mock_result(self.nodes[1], '0 {"invalid json"}')
```

💬 Eunovo commented on pull request "validation: ensure assumevalid is always used during reindex":
(https://github.com/bitcoin/bitcoin/pull/31615#issuecomment-3095873057)
Rebased on master@ https://github.com/bitcoin/bitcoin/commit/5878f35446

👍 dergoegge approved a pull request: "fuzz: Make process_message(s) more deterministic"
(https://github.com/bitcoin/bitcoin/pull/32822#pullrequestreview-3037221638)
reACK fa1a14a13a15ecfb7587a94ee86b4ace7c819519

💬 Eunovo commented on pull request "Silent Payments: Receiving":
(https://github.com/bitcoin/bitcoin/pull/32966#discussion_r2218671774)
IIUC The silent payments wallet needs to scan entire blocks since it doesn't know ahead of time which `scriptPubKeys` to use in a GCS filter. Since we can't skip blocks, fast rescans shouldn't provide any advantages here.

💬 stickies-v commented on pull request "p2p: rename GetAddresses -> GetAddressesUncached":
(https://github.com/bitcoin/bitcoin/pull/32994#discussion_r2218745362)
> I'm wondering if it makes more sense to signal the lack of something ... instead of the additional functionality in the pair

I considered that too, but I don't think that's better. The "Unchached" one is unsafe, and for that it should stand out. My initial thought was to name it `GetAddressesUnsafe` or `GetAddressesInternal`, but those names carry less meaning, so I eventually landed on just describing what it does, which I think at least gives people somewhat familiar with the code a good
...

📝 maflcko opened a pull request: "NOMERGE DEBUG WIP ignore"
(https://github.com/bitcoin/bitcoin/pull/33028)

💬 fanquake commented on pull request "ci: Use APT_LLVM_V in msan task":
(https://github.com/bitcoin/bitcoin/pull/32999#issuecomment-3096172593)
Should probably adjust the timeout / comment:
https://github.com/bitcoin/bitcoin/blob/5878f35446ae260ccb0ab5051b795b4364f77951/.cirrus.yml#L160
but given it's being moved in #32989, left a note there.