Skip to content
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

Add RPC to bump local commit fees #2743

Merged
merged 1 commit into from Sep 14, 2023
Merged

Add RPC to bump local commit fees #2743

merged 1 commit into from Sep 14, 2023

Conversation

t-bast
Copy link
Member

@t-bast t-bast commented Sep 13, 2023

Node operators may disable automatic fee-bumping on their local commit if they don't have anything at stake (no pending HTLCs), which saves fees in most cases. A drawback in that case is that if the commitment doesn't confirm quickly enough, the remote's funds are also locked.

This can be an issue for LSPs, where the remote peer doesn't have a good ability to fee-bump commit txs. We give more control to the node operator by letting them fee-bump local commit txs explicitly through the RPC to unblock wallet users funds.

@t-bast t-bast requested a review from pm47 September 13, 2023 15:08
Node operators may disable automatic fee-bumping on their local commit if
they don't have anything at stake (no pending HTLCs), which saves fees in
most cases. A drawback in that case is that if the commitment doesn't
confirm quickly enough, the remote's funds are also locked.

This can be an issue for LSPs, where the remote peer doesn't have a good
ability to fee-bump commit txs. We give more control to the node operator
by letting them fee-bump local commit txs explicitly through the RPC to
unblock wallet users funds.
@codecov-commenter
Copy link

Codecov Report

Merging #2743 (a012277) into master (404e3d5) will increase coverage by 0.01%.
Report is 1 commits behind head on master.
The diff coverage is 65.62%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master    #2743      +/-   ##
==========================================
+ Coverage   85.87%   85.88%   +0.01%     
==========================================
  Files         215      215              
  Lines       17833    17854      +21     
  Branches      775      756      -19     
==========================================
+ Hits        15314    15334      +20     
- Misses       2519     2520       +1     
Files Changed Coverage Δ
...r/acinq/eclair/blockchain/fee/OnChainFeeConf.scala 76.47% <0.00%> (-10.20%) ⬇️
...in/scala/fr/acinq/eclair/channel/ChannelData.scala 100.00% <ø> (ø)
.../fr/acinq/eclair/channel/publish/TxPublisher.scala 97.50% <ø> (ø)
...in/scala/fr/acinq/eclair/channel/fsm/Channel.scala 85.28% <42.85%> (-0.48%) ⬇️
...r-core/src/main/scala/fr/acinq/eclair/Eclair.scala 58.66% <100.00%> (+1.00%) ⬆️
...c/main/scala/fr/acinq/eclair/channel/Helpers.scala 94.73% <100.00%> (-0.16%) ⬇️

... and 7 files with indirect coverage changes

@t-bast t-bast merged commit 148fc67 into master Sep 14, 2023
1 check passed
@t-bast t-bast deleted the bump-commit-rpc branch September 14, 2023 15:07
t-bast added a commit that referenced this pull request Feb 29, 2024
This release introduces a few API changes:

- `bumpforceclose` can be used to make a force-close confirm faster, by
  spending the anchor output (#2743)
- `open` now takes an optional parameter `--fundingFeeBudgetSatoshis` to
  define the maximum acceptable value for the mining fee of the funding
  transaction. This mining fee can sometimes be unexpectedly high
  depending on available UTXOs in the wallet (#2808)
- `rbfopen` now takes a mandatory parameter `--fundingFeeBudgetSatoshis`
  with the same semantics as for `open` (#2808)
- `payoffer` can be used to pay experimental Bolt 12 offers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants