Skip to content

Add new plugin bumpit #672

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ca-ruz
Copy link
Contributor

@ca-ruz ca-ruz commented Apr 28, 2025

A Core Lightning plugin to bump an open channel transaction by creating a CPFP, useful when the funding transaction is stuck in the mempool or has a low fee.

https://github.com/ca-ruz/bumpit

Note: This plugin does not work with lightning version 24.08 because it uses the listaddresses rpc call, which was introduced in version 24.11.

@ca-ruz ca-ruz force-pushed the bumpchannelopen branch 3 times, most recently from c572a67 to 1a9e95e Compare April 29, 2025 01:16
@ca-ruz ca-ruz changed the title Add new plugin bumpchannelopen Add new plugin bumpit Apr 29, 2025
@ca-ruz ca-ruz marked this pull request as draft April 29, 2025 01:20
@ca-ruz ca-ruz marked this pull request as ready for review April 30, 2025 23:56
@chrisguida
Copy link
Collaborator

@ca-ruz looks great so far! I think the testing around the emergency reserve could be improved.. there is an edge case where the user creates a channel and then their change output becomes the emergency reserve, and if they bump this output with too high a fee, the emergency reserve could end up getting inadvertently spent. Please either add a very loud warning about this in the README and in the RPC help strings, or add some logic that removes this footgun.

@ca-ruz ca-ruz force-pushed the bumpchannelopen branch from 83e4695 to 4a94337 Compare June 4, 2025 19:49
@ca-ruz
Copy link
Contributor Author

ca-ruz commented Jun 4, 2025

@chrisguida Hey! Thanks for the feedback. I added logic that prevents the user from spending the emergency reserve and added a couple of other features. I also updated the tests, all should be passing now.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants