Specify fees explicitly via JSON-RPC #570

pull luke-jr wants to merge 4 commits into bitcoin:master from luke-jr:force_send changing 7 files +92 −34
  1. luke-jr commented at 7:51 PM on October 5, 2011: member

    Disables automatically adding "minimum" fees for JSON-RPC methods-- instead, it returns an error or, iff the user sets the new second parameter "force" to the 'settxfee' JSON-RPC call, sends the transaction with the user-specified fee. This second parameter to 'settxfee' is only enabled if bitcoind is started with the undocumented -nosafefees option.

    Currently 4/6 support this change: https://bitcointalk.org/index.php?topic=46925

  2. gavinandresen commented at 1:27 AM on December 1, 2011: contributor

    No consensus on this-- and it removes a feature that I think most web services want ("just send whatever fee is likely to get the damn transactions into blocks, I'll eat the costs because they're so small anyway" -- anyway, that was what I wanted for ClearCoin).

  3. gavinandresen closed this on Dec 1, 2011

  4. gavinandresen reopened this on Dec 1, 2011

  5. luke-jr commented at 5:51 PM on December 2, 2011: member

    With this part, automatic fees up to "maxtxfee" (default 0.01 BTC) will be accepted for sends without further JSON-RPC interaction.

  6. sipa commented at 2:44 AM on January 2, 2012: member

    ACK on the -maxtxfee idea; I'll check the code soon.

    About -nosafefees - yes, but only after some way of reverting transactions that do not get accepted by the network is present.

  7. TheBlueMatt commented at 5:50 AM on January 2, 2012: member

    Seriously, the upgrading transactions stuff was already coded, all you have to do is implement it and do it well...

  8. jgarzik commented at 10:30 PM on February 3, 2012: contributor

    Disagree with "accept any transaction...from myself" because that is a useful safety net.

    No opinion on the other stuff. Probably should admin-close and let this issue/code sit until fees are straightened out.

  9. luke-jr commented at 10:50 PM on February 3, 2012: member

    Re "accept any transaction...from myself": By the time they're being rejected, it's already too late to undo them.

  10. jgarzik commented at 9:07 PM on May 8, 2012: contributor

    I think this is a mess we don't want to merge... NAK from me at least.

    If and when fees get redone, this code will change anyway. Until that time, it is debatable that these will be used by >1 users.

  11. luke-jr commented at 9:09 PM on May 8, 2012: member

    Someone else is maintaining a complete fork just for this, and it seems to have a >1 userbase: https://bitcointalk.org/index.php?topic=22434.0

  12. sipa commented at 9:18 PM on May 8, 2012: member

    I'm quite sure some people are interested in this, especially the the ability to prevent transaction from taking a unexpected fee without interaction. There are alternatives though, like an RPC to prepare a transaction, and allow inspection before submitting it.

    The nosafefees part... sure, one day, but we definitely need saner behaviour when non-accepting / conflicting transactions exist in a wallet.

  13. Don't automatically include fees via JSON-RPC, and (with undocumented -nosafefees option) allow forcing them to send with under the 'minimum' 96f3257234
  14. Accept automatic fees up to new "maxtxfee" parameter 911e77253e
  15. Accept any transaction (fee-free or even non-standard) from myself 6a0a25faa4
  16. Refactor maxtxfee and -nosafefees slightly to work together de4a448896
  17. jgarzik commented at 4:05 PM on June 27, 2012: contributor

    Do we still need this, in light of the raw transaction stuff in #1456 ?

  18. luke-jr commented at 4:12 PM on June 27, 2012: member

    I think so. Just because one can implement the same behaviour by implementing their own Bitcoin transaction maker externally, doesn't mean the simpler use-cases shouldn't be supported for software that just wants to deal with the financial aspects.

  19. jgarzik commented at 3:34 PM on August 1, 2012: contributor

    No consensus, it mixes in fee behavior which is actively debated, and this pull request has become "all over the place"

    Closing, but you are welcome to re-open smaller, more fine-grained changes as separate pull requests that might be more easily ACK'd or NAK'd.

  20. jgarzik closed this on Aug 1, 2012

  21. ptschip referenced this in commit e06e4efb99 on May 15, 2017
  22. ptschip referenced this in commit 28a7959f5c on May 19, 2017
  23. KolbyML referenced this in commit bc86927231 on Sep 4, 2020
  24. DrahtBot 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-04-14 15:16 UTC

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