Add RPC call estimatetxfee #283

pull sassame wants to merge 0 commits into bitcoin:master from sassame:estimatetxfee changing 0 files +0 −0
  1. sassame commented at 2:24 AM on May 29, 2011: none

    This change is per discussion at http://forum.bitcoin.org/index.php?topic=8153.0;

    Added ability to estimate the transaction fee for a given transfer amount. Use case concerns transferring BTC out of an "account", and being able to determine if the account has enough BTC to cover transfer + fees.

  2. sipa commented at 11:51 AM on May 29, 2011: member

    The problem is that some (intended) indetermism is present while generating a transaction, which may influence the size of a transactions and thus its fee according to a certain policy. I believe the correct way should be that payment RPC's take an additional argument that states that no actual payment only the creation of a transaction is required. The created transaction can then be inspected, and through another call rejected or accepted.

  3. TheBlueMatt commented at 12:11 PM on May 29, 2011: member

    Can you rebase this into one commit? Also, I agree with sipa on the proper way to do this. I'm hoping someone redoes all the UI for transaction fees for 0.4.0.

  4. sassame commented at 6:31 PM on May 29, 2011: none

    Sipa - so, you're suggesting we update the send payment RPCs to return transaction details, instead of just the Transaction ID, then include a new flag that indicates a fake transaction? Or, would we have a non-committed transaction that I could still query by txn ID?

    Alternatively, we could keep the estimate call, then include a "max tx fee" parameter on the send payment RPC, so it fails if the tx fee ends up over the given amount. From my perspective, I'm always making these calls from inside a transaction, so I don't mind a later call failing, since it's easy for me to re-run the whole transaction.

    Finally, I'm new to git, how do I rebase? Just create a new branch and make all the changes in one push?

  5. sassame merged this on May 29, 2011
  6. sassame closed this on May 29, 2011

  7. sassame commented at 10:31 PM on May 29, 2011: none

    Accidentally closed while trying to rebase, opening new pull request.

  8. sipa referenced this in commit 7863d105ea on Aug 4, 2015
  9. sipa referenced this in commit 780d5b3c7b on Aug 23, 2015
  10. TheBlueMatt referenced this in commit a671356e1f on Oct 20, 2015
  11. sipa referenced this in commit 9177950c74 on Oct 21, 2015
  12. sipa referenced this in commit f4787d1caf on Oct 21, 2015
  13. sipa referenced this in commit 6557a8cd46 on Oct 26, 2015
  14. sipa referenced this in commit ea06490d14 on Oct 27, 2015
  15. sipa referenced this in commit 003bb87153 on Nov 5, 2015
  16. sipa referenced this in commit bfd83199c3 on Nov 11, 2015
  17. sipa referenced this in commit b437ea7ec9 on Nov 12, 2015
  18. sipa referenced this in commit 1d84107924 on Nov 12, 2015
  19. rebroad referenced this in commit 40ead34fbe on Dec 7, 2016
  20. deadalnix referenced this in commit 432e1ceca0 on Jan 19, 2017
  21. ptschip referenced this in commit e978dd1ddf on Feb 9, 2017
  22. classesjack referenced this in commit 763cfc7791 on Jan 2, 2018
  23. lateminer referenced this in commit ef0ced79b1 on Oct 16, 2019
  24. attilaaf referenced this in commit 93283799cb on Jan 13, 2020
  25. hebasto referenced this in commit 328da33557 on Apr 28, 2021
  26. 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-05-02 12:16 UTC

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