problematic error reporting in sendtoaddress #211

issue mikegogulski opened this issue on May 9, 2011
  1. mikegogulski commented at 4:08 PM on May 9, 2011: none
    bitcoin@btcserver:~/bitcoin-0.3.21/bin/32$ ./bitcoind sendtoaddress 16bRQcbr1VcxxxxxZQxqGhi242eL 0.2 
    error: {"code":-4,"message":"Error: This transaction requires a transaction fee of at least 0.01 because of its amount, complexity, or use of recently received funds  "}
    

    Reminds me a bit of "Password incorrect or domain controller unavailable".

    My "Errors and Error Messages: Handbook for Dungeon Masters" guidebook says:

    §16(a)(vii): "Each and every error message shall describe the one, single, sole, only and unique error condition generating the message."

    §16(a)(xix): "No error message shall contain any of the words 'or', 'nor', 'some', 'any', 'several', 'either' or 'neither', nor any of the collocations 'one of', 'some of', 'one or more of', and so on."

  2. sipa commented at 11:08 PM on February 21, 2012: member

    Would a "Anti-spam rules require that this transactions carries a transaction fee" be better?

    It's impossible to split out the different cases, as it is rather "some complex combination of" rather than "or".

  3. sipa cross-referenced this on Feb 21, 2012 from issue "Over the Size Limit" Should be "Under/Over" by ussjoin
  4. atum commented at 8:17 AM on April 10, 2012: none

    This error message is also problematic because it is triggered for spending coins with too few confirmations as well. Users are not made aware of the fact that they could avoid the fee by just waiting a bit longer. It would probably make sense to use a different message in that case.

  5. enterprisey commented at 9:04 PM on April 25, 2013: contributor

    @sipa That would be a good idea.

  6. laanwj commented at 1:47 PM on February 17, 2014: member

    The error occurs only when the value of the transaction + the fee exceeds the balance.

    https://github.com/bitcoin/bitcoin/blob/master/src/wallet.cpp#L1543

    What about removing the explanatory fluff and making it just: "The transaction amount with fee added exceeds the spendable balance"?

  7. zathras-crypto referenced this in commit 29e8fe6f3f on Dec 4, 2014
  8. paveljanik cross-referenced this on Oct 1, 2015 from issue allow RPC send commands to specify TX fee per TX by mikegogulski
  9. laanwj commented at 4:28 PM on February 9, 2016: member

    As it is a bit absurd to have an issue about a message change open for 5 years, I'm going to close this issue. I haven't ever heard anyone else complain about this message. And yes, it's still there.

  10. laanwj closed this on Feb 9, 2016

  11. laanwj cross-referenced this on Oct 26, 2016 from issue rpc: Remove invalid explanation from wallet fee message by laanwj
  12. deadalnix referenced this in commit db059a6f61 on Jan 6, 2017
  13. lateminer referenced this in commit 7147c3b8a8 on Dec 9, 2017
  14. classesjack referenced this in commit a633d64186 on Jan 2, 2018
  15. MarcoFalke referenced this in commit 8ca6bd0dac on Feb 25, 2021
  16. rajarshimaitra referenced this in commit 453606a107 on Aug 5, 2021
  17. bitcoin locked this on Sep 8, 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-05-20 06:56 UTC