policy: Remove individual transaction <minrelay restriction #33892

pull instagibbs wants to merge 2 commits into bitcoin:master from instagibbs:2025-11-remove_singletx_minrelay_req changing 7 files +31 −53
  1. instagibbs commented at 8:24 pm on November 17, 2025: member

    Prior to cluster mempool, a policy was in place that disallowed non-TRUC transactions from being TX_RECONSIDERABLE in a package setting if it was below minrelay. This was meant to simplify reasoning about mempool trimming requirements with non-trivial transaction topologies in the mempool. This is no longer a concern post-cluster mempool, so this is relaxed.

    In effect, this makes 0-value parent transactions relayable through the network without the TRUC restrictions and thus the anti-pinning protections.

  2. DrahtBot added the label TX fees and policy on Nov 17, 2025
  3. DrahtBot commented at 8:24 pm on November 17, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33892.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    No conflicts as of last run.

  4. DrahtBot added the label Needs rebase on Nov 25, 2025
  5. instagibbs force-pushed on Nov 25, 2025
  6. instagibbs marked this as ready for review on Nov 25, 2025
  7. DrahtBot removed the label Needs rebase on Nov 25, 2025
  8. policy: Allow any transaction version with < minrelay
    Prior to cluster mempool, a policy was in place that
    disallowed non-TRUC transactions from being
    TX_RECONSIDERABLE in a package setting if it was below
    minrelay. This was meant to simplify reasoning about mempool
    trimming requirements with non-trivial transaction
    topologies in the mempool. This is no longer a concern
    post-cluster mempool, so this is relaxed.
    
    In effect, this makes 0-value parent transactions relayable
    through the network without the TRUC restrictions and
    thus the anti-pinning protections.
    1488315d76
  9. instagibbs force-pushed on Nov 26, 2025
  10. add release note about supporing non-TRUC <minrelay txns 22b7a25c4a
  11. in doc/release-notes-33892.md:4 in fda7a16c02
    0@@ -0,0 +1,5 @@
    1+P2P and network changes
    2+-----------------------
    3+
    4+- All transaction versions includeing anon-TRUC can now be relayed when below minrelay,
    


    fanquake commented at 2:38 pm on November 26, 2025:
    LLM: s/includeing/including/
  12. instagibbs force-pushed on Nov 26, 2025
  13. in test/functional/mempool_ephemeral_dust.py:210 in 1488315d76 outdated
    206@@ -207,18 +207,16 @@ def test_nonzero_dust(self):
    207         self.connect_nodes(0, 1)
    208         assert_mempool_contents(self, self.nodes[0], expected=[])
    209 
    210-    # N.B. If individual minrelay requirement is dropped, this test can be dropped
    


    kevkevinpal commented at 6:29 pm on November 26, 2025:
    why not drop the whole test as per this comment?

    instagibbs commented at 4:26 pm on December 1, 2025:
    changed my mind; it’s good to have v2 ephemeral dust coverage since we don’t have much/any?
  14. fanquake added this to the milestone 31.0 on Dec 4, 2025

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-12-17 06:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me