✅ fanquake closed a pull request: "BIP324: CKey encode/decode to elligator-swift"
(https://github.com/bitcoin/bitcoin/pull/23432)

👍 vasild approved a pull request: "p2p: skip netgroup diversity follow-up"
(https://github.com/bitcoin/bitcoin/pull/27467#pullrequestreview-1389339459)
ACK 11bb31c1c43b5da36ca8509b5747abfb3278ffcd

📝 willcl-ark opened a pull request: "doc: Improve documentation of rpcallowip"
(https://github.com/bitcoin/bitcoin/pull/27480)
Closes #21070

v21.0 introduced a behaviour changed noted in #21070 where using a config value `rpcallowip=::0` no longer also permitted ipv4 ip addresses.

The rpc_bind.py functional test covers this new behaviour already by checking that the list of bind addresses exactly matches what is expected so this commit only updates the documentation.

💬 ajtowns commented on pull request "mempool: keep CPFP'd transactions when loading from mempool.dat":
(https://github.com/bitcoin/bitcoin/pull/27476#issuecomment-1512510817)
Maybe move most of the text in the PR description into a comment? "Review this first" and "Alternatives considered" stuff is useful here, but doesn't seem useful for the merge commit message?

⚠️ adityachoubeyy opened an issue: "UD"
(https://github.com/bitcoin/bitcoin/issues/27481)
### Motivation

UNIDENTIFIED SIMULATION DISTINCTION

### Possible solution

N/A

### Useful Skills

Unique Unidentified Distinction

### Guidance for new contributors

Want to work on this issue?

For guidance on contributing, please read [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) before opening your pull request.

✅ fanquake closed an issue: "UD"
(https://github.com/bitcoin/bitcoin/issues/27481)

:lock: fanquake locked an issue: "UD"
(https://github.com/bitcoin/bitcoin/issues/27481)

👍 vasild approved a pull request: "test: add regression tests for #27468 (invalid URI segfaults)"
(https://github.com/bitcoin/bitcoin/pull/27477#pullrequestreview-1389480071)
ACK 6a77d290da589bd5620585def5bfc019e242e189

💬 fanquake commented on pull request "addrman: Enable selecting addresses by network":
(https://github.com/bitcoin/bitcoin/pull/27214#issuecomment-1512672126)
This will be merged after branch-off. It would be good to either get the follow up opened in advance, so we can avoid any time between merging, and the potential for intermittent CI failures, or, if that change is straight-forward enough, you could push an additional commit here, leaving the rest of the branch as-is.

💬 vasild commented on issue "Add per message stats to getnettotals rpc":
(https://github.com/bitcoin/bitcoin/issues/26337#issuecomment-1512696265)
@satsie, thanks, all looks good except the semantic of the argument (I have not looked at the code yet).

To me "aggregate by" means to "sum by" those fields, similarly to the `GROUP BY` clause in SQL. So, if I aggregate by `connection_type` and `message_type` I would expect to see only `network` and `direction` in the output. I do not have a strong opinion whether the argument should act that way or in the inverse way, but the name of the argument should be chosen in such a way as to avoid co
...

💬 fanquake commented on pull request "BIP324: Handshake prerequisites":
(https://github.com/bitcoin/bitcoin/pull/23561#issuecomment-1512705307)
Closing this for now, as it's partly included in #27479.

✅ fanquake closed a pull request: "BIP324: Handshake prerequisites"
(https://github.com/bitcoin/bitcoin/pull/23561)

💬 fanquake commented on pull request "test: add regression tests for #27468 (invalid URI segfaults)":
(https://github.com/bitcoin/bitcoin/pull/27477#issuecomment-1512724993)
This has been merged.

✅ fanquake closed a pull request: "test: add regression tests for #27468 (invalid URI segfaults)"
(https://github.com/bitcoin/bitcoin/pull/27477)

💬 TheCharlatan commented on pull request "blockstorage: Adjust fastprune limit if block exceeds blockfile size":
(https://github.com/bitcoin/bitcoin/pull/27191#discussion_r1169737011)
I checked this carefully again and don't think anymore that the `+ 1` is necessary.

👍 hebasto approved a pull request: "[24.x] Additional backports for 24.1"
(https://github.com/bitcoin/bitcoin/pull/27474#pullrequestreview-1389688800)
ACK 5c937ae7c323198c76f1743870c68f8576305a6e

💬 fanquake commented on pull request "ci: Use Cirrus CI dockerfile env":
(https://github.com/bitcoin/bitcoin/pull/27340#issuecomment-1512764647)
Going to merge this now, so that `25.x` and `master` will have the same CI infra.

🚀 fanquake merged a pull request: "ci: Use Cirrus CI dockerfile env"
(https://github.com/bitcoin/bitcoin/pull/27340)

💬 stickies-v commented on pull request "[24.x] Additional backports for 24.1":
(https://github.com/bitcoin/bitcoin/pull/27474#issuecomment-1512851845)
Suggest adding https://github.com/stickies-v/bitcoin/commit/db6cf79694112a4679378fdd1f2bdf69cf6adbea which backports #27468, but without the changes to `rest_mempool` (query parameters in this endpoint were added only after v24 in #26207).

📝 fanquake opened a pull request: "kernel: chainparams updates for 25.x"
(https://github.com/bitcoin/bitcoin/pull/27482)
Update chainparams pre `25.x` branch off.
Co-Author in the commits as a PR (#27223) had previously been opened too-early to do the same.