policy: remove incorrect `MANDATORY_SCRIPT_VERIFY_FLAGS` comment #34957

pull ismaelsadeeq wants to merge 1 commits into bitcoin:master from ismaelsadeeq:03-2026-fix-incorrect-policy-comment changing 1 files +1 −2
  1. ismaelsadeeq commented at 2:17 PM on March 30, 2026: member

    Since #33050

    This comment claiming that failing mandatory script checks may trigger a DoS ban/punishment is false. CheckInputScript also dispute this comment.

    https://github.com/bitcoin/bitcoin/blob/b0f68f0a3a361c62cb1e1cb49aacaeac171f3079/src/validation.cpp#L2113-L2118

  2. policy: remove incorrect MANDATORY_SCRIPT_VERIFY_FLAGS comment
    The claim that failing mandatory script checks may trigger a DoS ban
    is incorrect; Bitcoin Core does not automatically ban peers for
    violating any of these flag checks.
    5fa6898818
  3. DrahtBot added the label TX fees and policy on Mar 30, 2026
  4. DrahtBot commented at 2:17 PM on March 30, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK darosior, instagibbs

    If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. darosior approved
  6. darosior commented at 2:41 PM on March 30, 2026: member

    utACK 5fa6898818c3fa0000c5ce7b070b67b4277642a3

    We don't disconnect/discourage peers sending an unconfirmed transaction failing consensus rules anymore since v30 (#33050).

  7. fanquake commented at 3:40 PM on March 30, 2026: member
  8. instagibbs approved
  9. instagibbs commented at 12:23 PM on March 31, 2026: member

    ACK 5fa6898818c3fa0000c5ce7b070b67b4277642a3

  10. fanquake merged this on Mar 31, 2026
  11. fanquake closed this on Mar 31, 2026

  12. ismaelsadeeq deleted the branch on Mar 31, 2026
  13. ajtowns commented at 7:29 AM on April 1, 2026: contributor

    Post-merge utACK


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: 2026-04-26 06:12 UTC

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