BIP-352: warn against stopping scan due to wallet policy filtering #2134

pull Eunovo wants to merge 1 commits into bitcoin:master from Eunovo:always-scan-after-a-match changing 1 files +1 −1
  1. Eunovo commented at 9:39 AM on April 7, 2026: contributor

    From the discussion in #2106 (comment), this PR adds a warning to the "if no matches are found, stop" scanning step. Without it, wallet developers may be tempted to apply policy filtering (e.g. dust) before deciding to continue, causing subsequent outputs for the same sender to be missed.

  2. BIP-352: warn against stopping scan due to wallet policy filtering
    Adds a warning to the "if no matches are found, stop" scanning
    step. Without it, wallet developers may be tempted to apply policy
    filtering (e.g. dust) before deciding to continue,
    causing subsequent outputs for the same sender to be missed.
    df87830829
  3. Eunovo commented at 9:40 AM on April 7, 2026: contributor

    cc: @theStack

  4. murchandamus added the label Proposed BIP modification on Apr 7, 2026
  5. murchandamus added the label Pending acceptance on Apr 7, 2026
  6. murchandamus commented at 3:28 PM on April 7, 2026: member

    LGTM, waiting for @theStack to review, though.

  7. theStack commented at 5:16 PM on April 7, 2026: contributor

    lgtm ACK df8783082922d45ea29e40e590f5a71e4397af34

    Thanks!

  8. murchandamus merged this on Apr 7, 2026
  9. murchandamus closed this on Apr 7, 2026

  10. murchandamus removed the label Pending acceptance on Apr 7, 2026

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-14 11:10 UTC

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