💬 fjahr commented on pull request "test, assumeutxo: Add test to ensure failure when mempool not empty":
(https://github.com/bitcoin/bitcoin/pull/29394#discussion_r1483610267)
Actually, when you switch to use node 0 you can remove this line entirely because node 0 is not used to load snapshots in the rest of the test.

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483614824)
Hmmm I find early returns in the middle of a function a bit brittle. Would be easier to write a bug if e.g. we added another v3 rule at the bottom of the function.

💬 fjahr commented on pull request "test, assumeutxo: Add test to ensure failure when mempool not empty":
(https://github.com/bitcoin/bitcoin/pull/29394#discussion_r1483614887)
@hernanmarino you marked the other thread as resolved while I was writing, this change works and I think that's what @maflcko had in mind: https://github.com/fjahr/bitcoin/commit/a6883b8919d7348bab56f6ebee0818912146ae52

💬 instagibbs commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483617248)
fair enough, I just keep forgetting I can assume it has a parent because this block is pretty large. maybe a skill issue on my part

💬 dergoegge commented on pull request "RFC: build: remove confusing and inconsistent disable-asm option":
(https://github.com/bitcoin/bitcoin/pull/29407#issuecomment-1934972190)
@jonatack see https://github.com/bitcoin/bitcoin/issues/29178 (we prefer having this broken over reverting the change that introduced it for some reason)

I'm not sure if the `random` harness would trigger the ASan error, the docs mention sha256 which `random` does not use afaict. Could you try `txorphan` or a different harness that does some hashing?

Did you set `UBSAN_OPTIONS` to include `halt_on_error=1`? that is likely why it keeps running if you didn't

💬 brunoerg commented on pull request "i2p: log connection was refused due to arbitrary port":
(https://github.com/bitcoin/bitcoin/pull/29393#issuecomment-1934972685)
Force-pushed addressing https://github.com/bitcoin/bitcoin/pull/29393#discussion_r1483580775

💬 brunoerg commented on pull request "p2p: Allow whitelisting manual connections":
(https://github.com/bitcoin/bitcoin/pull/27114#discussion_r1483623515)
Sounds good, I'll address it.

💬 brunoerg commented on pull request "p2p: Allow whitelisting manual connections":
(https://github.com/bitcoin/bitcoin/pull/27114#discussion_r1483624485)
I'll fix it

💬 jonatack commented on pull request "i2p: log connection was refused due to arbitrary port":
(https://github.com/bitcoin/bitcoin/pull/29393#issuecomment-1934983441)
ACK 5b358cdd1a5f5d2fe87a9e41c638996eab2e2796

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628446)
done

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628863)
added

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628734)
added a comment

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628675)
removed

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628493)
done

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483628403)
Gonna mark this as resolved 👍

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#discussion_r1483631193)
I agree that in the case above D shouldn't need to conflict with all children, so will leave as is.

💬 glozow commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#issuecomment-1934997643)
> To unblock progress on other work, my suggestion is that we do the following:
>
> * modify this PR to introduce the v3 policy and validation rules, but not make v3 transactions standard for relay yet -- this should be a very small code change I think

Pushed to address @instagibbs review and make v3 not-yet-standard for mainnet.

💬 ariard commented on pull request "v3 transaction policy for anti-pinning":
(https://github.com/bitcoin/bitcoin/pull/28948#issuecomment-1935034851)
> To be clear, package relay with package RBF is meant to be resistant to these kinds of network topology aware (NTA) pinning scenarios. With correctly implemented package relay it doesn't matter that a CPFP tx did a CPFP on top of a commitment transaction that no-one but the originator has: provided the fee (or with RBFR, fee-rate) outbids the competing commitment transaction, it will be replaced as nodes evaluate the new package as a whole.

i’m not sure if the adversary injects malicious pa
...

🤔 pablomartin4btc reviewed a pull request: "release: Update translations for v27.0 soft translation string freeze"
(https://github.com/bitcoin/bitcoin/pull/29397#pullrequestreview-1871391193)
post-merge tACK 71927b24e5aceecd8a07cdaeb916898d45486bea

I've tested it following the [translate process guide](https://github.com/bitcoin/bitcoin/blob/master/doc/translation_process.md) (which `Transifex` setup is outdated and needs to be corrected) on master and compared the resulted changes against this PR verifying there was no differences.

💬 achow101 commented on pull request "test: fix intermittent failure in `rpc_setban.py --v2transport`, run it in CI":
(https://github.com/bitcoin/bitcoin/pull/29372#issuecomment-1935056868)
ACK cc87ee4c3934028e78a59de509951ff7226ec80d