settxfee 0.00000000 returns true #18315

issue mrbabtc opened this issue on March 11, 2020
  1. mrbabtc commented at 2:20 PM on March 11, 2020: none

    No description provided.

  2. mrbabtc added the label Bug on Mar 11, 2020
  3. MarcoFalke added the label RPC/REST/ZMQ on Mar 11, 2020
  4. MarcoFalke added the label Wallet on Mar 11, 2020
  5. MarcoFalke added the label Brainstorming on Mar 11, 2020
  6. fjahr commented at 9:52 PM on March 29, 2020: member

    You did not provide what the behavior is that you would have expected from settxfee. So I am not sure if you were aware, but it is a feature of settxfee that you can deactivate that fee parameter by passing in 0. true is correct as a return value here since the call succeeds. But I agree that this feature is not documented properly and also the return value can be confusing, even if you are aware of the feature. I am trying to improve both of these issues in #18467.

  7. MarcoFalke commented at 10:06 PM on March 29, 2020: member

    I think another issue is that settxfee 1111111 returns true :man_shrugging:

  8. fjahr commented at 10:28 PM on March 29, 2020: member

    I think another issue is that settxfee 1111111 returns true 🤷‍♂

    Why stop there? src/bitcoin-cli settxfee 21000000 returns true as well ;)

    Yes, there is not any upper bound to the fee rate you can set aside from total supply. Is there already precedent for restricting that user choice? I could see that we give a warning that the fee appears to be high but I am not sure what value to compare this to. Since this is a persistent setting I don't think any current dynamic estimate is a good reference.

  9. MarcoFalke commented at 10:52 PM on March 29, 2020: member

    I think there is some kind of -maxtxfee wallet command line setting

  10. fjahr commented at 8:02 PM on March 30, 2020: member

    I think there is some kind of -maxtxfee wallet command line setting

    Good idea, I have used this now.

  11. fjahr commented at 2:22 PM on April 14, 2020: member

    @mrbabtc could you take a look at #18467 and let me know if this fixes your issue? It does not change the return value but improves documentation about the behavior.

  12. MarcoFalke commented at 1:24 PM on April 22, 2020: member

    Anything left to do here?

  13. MarcoFalke closed this on Apr 22, 2020

  14. mrbabtc commented at 3:28 PM on April 24, 2020: none

    @mrbabtc could you take a look at #18467 and let me know if this fixes your issue? It does not change the return value but improves documentation about the behavior.

    Sorry , I wasnt checking it And I think that settxfee 0 possible could be confusing because min relay fee is 1 sat/byte

  15. DrahtBot locked this on Feb 15, 2022

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

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