-
Notifications
You must be signed in to change notification settings - Fork 103
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
Manual fee rate support #595
Conversation
log.Infof("sending with manual fee rate") | ||
|
||
default: | ||
feeRate, err = p.cfg.ChainBridge.EstimateFee( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than conditionally call estimation here, we could instead lift it up to the rpcserver. So then we can drop the pointer to fee rate, as we compute that head of time.
In this commit, we add support for overriding the fee estimator and providing a target fee rate to the planter when finalizing a batch. Provided fee rates are only checked against a minimum value and not a maximum fee rate.
In this commit, we add support for overriding the fee estimator and providing a target fee rate to the freighter when performing a transfer. This only applies to sends to addresses and not via interactive vPSBTs.
c009c5a
to
7bac238
Compare
7bac238
to
96c8cc3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🦚
In line with our other software, this flag could be called |
Good point, will add that change in the upcoming fee PR. |
Fixes #587
Fixes #538
Updates the planter and freighter to accept a fee rate in sat/kw, the same unit as the fee estimator, to use for the anchor transaction. The fee rate is checked against a minimum but not maximum, so users should be careful with fee units to avoid accidental overpayment.