-
Notifications
You must be signed in to change notification settings - Fork 37.7k
fees: Remove fallbackfee default #11918
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
fa33f64
to
faad60c
Compare
Concept ACK. I think this is is an obviously good idea. If no fee estimate can be made, users should set a manual fee. Hardcoding a fallback fee is not realistic anymore. |
+1 |
Concept ACK... the only bothering thing is, that regtest mode gets quite useless without passing in a -fallbackfee... and for that reason #11882 (removes default fallbackfee on mainnet only) may make more sense. |
My understanding is that regtest mode becomes more and more useless as the number of differences to mainnet increase. If we special case that on mainnet, there'd be no way to test other than running a mainnet node. Especially since we plan to rework the gui based on the fallback settings, it is important to be able to test in a non-mainnet setting. |
@MarcoFalke But how do you avoid anyone doing manual regtest avoiding passing -fallbackfee every time they launch? |
They could specify it in their bitcoin.conf, assuming that it exists. If no bitcoin.conf exists, the only way I see is by passing it in on the command line. |
Closing, since #11882 is now similar and there is little value in keeping both open. |
fallbackfee is impossible to keep up-to-date by hardcoding a default value. This removes the default value and hands the responsibility to the user to determine an appropriate fallbackfee in case fee estimates are not available.
Todo: