💬 Sjors commented on pull request "Stratum v2 connman":
(https://github.com/bitcoin/bitcoin/pull/30332#issuecomment-2228301130)
Removed the accidentally included `src/common/sv2_messages.cpp`.

💬 maflcko commented on pull request "wallet: Add scan_utxo option to getbalances RPC":
(https://github.com/bitcoin/bitcoin/pull/28930#issuecomment-2228318061)
Are you still working on this? Seems stale for more than half a year, at least.

💬 Sjors commented on pull request "Have createNewBlock() return a BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/30440#issuecomment-2228365527)
Also added `getCoinbaseTx()` and `getWitnessCommitmentIndex()` which are needed for the `NewTemplate` message.

Since this PR now contains Stratum v2 specific stuff anyway, I pulled in #30441. Still in the process of incorporating all this into #29432 to make sure I didn't miss anything.

💬 Sjors commented on pull request "[WIP] Add getCoinbaseMerklePath() to Mining interface":
(https://github.com/bitcoin/bitcoin/pull/30441#issuecomment-2228366141)
Folded into #30440.

✅ Sjors closed a pull request: "[WIP] Add getCoinbaseMerklePath() to Mining interface"
(https://github.com/bitcoin/bitcoin/pull/30441)

💬 darosior commented on pull request "fuzz: bound some miniscript operations to avoid fuzz timeouts":
(https://github.com/bitcoin/bitcoin/pull/30197#discussion_r1677742992)
I just went for changing only that which was required. I think of `int` as "default" and chose to only deviate from the default when needed.

💬 stratospher commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677771441)
Done.

💬 stratospher commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677772287)
> Ok, I see, the peer.wait_for_disconnect() immediately returns, as there is no connection yet, and doesn't do anything.


I meant it as a check so that we know if `self._transport` got reset to `None` at some point of time. (`self._transport` is set in `connection_made()` when the connection is opened irrespective of whether p2p connection remains connected or disconnected)

> Why is it required to set a timeout here at all? The log message should happen before the disconnect, which is wai
...

💬 maflcko commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677780013)
Nit: You can use the existing `bumpmocktime` helper?

💬 maflcko commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677787182)
Can you explain what this means? `sendSet` is a boolean, so what does "populated" mean? Also, when removing `peer2` the test works fine, no? What would the "wrong" error message be?

🤔 marcofleon reviewed a pull request: "fuzz: bound some miniscript operations to avoid fuzz timeouts"
(https://github.com/bitcoin/bitcoin/pull/30197#pullrequestreview-2177671042)
Code review ACK bc34bc288824978ef4b98e8802b47cb863c8a8c2. The added comments are useful, thanks for those. Tested on the three inputs in https://github.com/bitcoin/bitcoin/issues/28812 that caused the timeouts.

🚀 fanquake merged a pull request: "fuzz: bound some miniscript operations to avoid fuzz timeouts"
(https://github.com/bitcoin/bitcoin/pull/30197)

👍 hebasto approved a pull request: "fix: rendering an amp characters in the wallet name for QMenu"
(https://github.com/bitcoin-core/gui/pull/828#pullrequestreview-2177678028)
re-ACK 7581817826e000c4f0a505e7f2785a611cac8f55

💬 hebasto commented on pull request "fix: rendering an amp characters in the wallet name for QMenu":
(https://github.com/bitcoin-core/gui/pull/828#discussion_r1677806107)
Maybe
```suggestion
// The single ampersand in the menu item's text sets a shortcut for this item.
```
?

💬 stratospher commented on pull request "Add fuzz test for FSChaCha20Poly1305, AEADChacha20Poly1305":
(https://github.com/bitcoin/bitcoin/pull/28263#discussion_r1677813086)
done.

🤔 glozow reviewed a pull request: "net: fix race condition in self-connect detection"
(https://github.com/bitcoin/bitcoin/pull/30394#pullrequestreview-2177691583)
ACK 16bd283b3ad05daa41259a062aee0fc05b463fa6

💬 pablomartin4btc commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#issuecomment-2228495226)
> For instance, the tests that pass a wallet CLI option to a non-wallet CLI command (-generate) seem to be mixing issues under test...
>
> ```diff
> - # Single or Multi wallet modes don't matter here as -generate command validation is performed before we send to RPC
> - # Note also that -rpcwallet arg intentionally adds some confusion but could be any arg starting with "-"
> - self.log.info('Test -generate with nblocks and -rpcwallet afterwards')
> -
...

💬 stratospher commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677835214)
done.

💬 Sjors commented on pull request "Stratum v2 Template Provider (take 3)":
(https://github.com/bitcoin/bitcoin/pull/29432#issuecomment-2228520507)
I incorporated the interface changes from #30440. In particular I made sure we only call `getBlock()` in the context of a `RequestTransactionData` message. The avoids sending the full serialized block over the interface as much as possible.

💬 stratospher commented on pull request "test: Fix intermittent failure in p2p_v2_misbehaving.py":
(https://github.com/bitcoin/bitcoin/pull/30420#discussion_r1677845216)
if the test is run in a loop, sometimes `"socket no message in first %I seconds"` is displayed and sometimes `"V2 handshake timeout"` is displayed.

[if `sendSet` is not set, "socket no message in first %i seconds"](https://github.com/bitcoin/bitcoin/blob/262260ce1e919613ba60194a5861b0b0a51dfe01/src/net.cpp#L1970) is displayed since that check is above [the check for the handshake timeout](https://github.com/bitcoin/bitcoin/blob/262260ce1e919613ba60194a5861b0b0a51dfe01/src/net.cpp#L1985) in `I
...