Switch wallet default to reject too-long transaction chains for mempool #15547

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:walletreject_true changing 2 files +6 −6
  1. instagibbs commented at 1:46 PM on March 6, 2019: member

    The benefits seem to outweigh the drawbacks: I still semi-regularly see chain limits being hit and users getting scared that they have lost their funds.

  2. Switch wallet default to reject too-long transaction chains for mempool f61b3f8cce
  3. fanquake added the label Wallet on Mar 6, 2019
  4. gmaxwell commented at 12:42 AM on March 7, 2019: contributor

    Ugh, I really don't want to rehash the arguments given previously.

    If users are hitting the chain limits with any frequency they should be fixed-- their justification in the first place was that ignoring apparent attacks they weren't being hit. Failing to make payments due to inexplicable internal machinery when otherwise payment making never fails can lead to funds loss and accidental fraud.

  5. instagibbs closed this on Mar 28, 2019

  6. MarcoFalke commented at 5:39 PM on March 28, 2019: member

    Failing to make payments due to inexplicable internal machinery

    It will eventually fail regardless because the wallet doesn't count non-mempool change, no?

  7. DrahtBot locked this on Dec 16, 2021

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-21 15:14 UTC

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