Anthony Towns

ACK e504b1fa1fa4d014b329dea81bfdf1aa55db238f ; LGTM and just checking the 1-of-3 case seems fine

> The DNS seeds resolve to IPs of random peers. It's entirely possible (and apparently reality) that some of those host malware. It's entirely possible that one could be hosting...

> Should we consider outbound connections for weightening as well? If we're at ~100 inbounds and ~8 outbounds, the weighting doesn't make much difference -- it just means 108 inbounds...

> @amitiuttarwar > > > what information do you imagine makes a particular inbound slot more valuable to one node vs another? > > I think we're talking about different...

Concept ACK. Might be good to have some way of being able to configure where to connect to for sensitive relay, defaulting to tor/i2p, but so that you could perhaps...

> It is deprecated It's only been deprecated in master for a couple of months; better to wait for it to actually be deprecated in the current release for a...

> The deprecation was covered in https://bitcoinops.org/en/newsletters/2023/09/20/ and 26.0 was released a few weeks ago. Unless anyone heard someone complain, this seems good to move forward now? Given we're doing...

Isn't this check a little bit later than ideal? Wouldn't it make more sense to check both maxfeerate and maxburnamount as part of `signrawtransactionwith*` and `walletprocesspsbt` so that you never...

> [Binance Pool, 8.2%](https://mempool.space/tx/64a26f750c7c58812bd475d054a9aa05248b6a8a2d53c0db38c0624197a4c68a) This replaced 87f20bf4a8b71a255e47876ca7563cdcab1a830490a1b130bbe71c28d8da3884 which had a 3.14sat/vb feerate. statoshi reported minfees up to 3.09sat/vb in the period between the original propagating and the replacement being mined....

> Changing the `IsUnspendable` logic in general seems very brittle, as it obviously leads to nodes with diverging UTXO sets. > > now that many projects depend on a normalized...