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

Latest CLN defaults to wumbo now. PeerSwap's wumbo / large-channels check is broken. #261

Closed
grubles opened this issue Nov 29, 2023 · 1 comment · Fixed by #269
Closed

Comments

@grubles
Copy link
Collaborator

grubles commented Nov 29, 2023

$ lightning-cli --signet peerswap-swap-out 140490x1x0 246942696 lbtc                                                                                      
{
   "code": -1,
   "message": "swap amount is 246942696000: need to enable option '--large-channels' to swap amounts larger than 2^32 msat"
}
$ cat ~/.lightning/config
signet
plugin=/home/user/.lightning/signet/plugins/peerswap
daemon
log-file=logs
log-level=debug

large-channels is missing, so PeerSwap considers it turned off and won't allow the swap to continue past that check.

Even adding large-channels to the CLN config does not help.

Commenting this line out allows the swap to continue:

return fmt.Errorf("swap amount is %d: need to enable option '--large-channels' to swap amounts larger than 2^32 msat", amtMsat)

@grubles grubles changed the title CLN defaults to wumbo now. PeerSwap's wumbo / large-channels check is broken. Latest CLN defaults to wumbo now. PeerSwap's wumbo / large-channels check is broken. Nov 29, 2023
@nepet
Copy link
Contributor

nepet commented Dec 1, 2023

My PR to upgrade cln on nix go merged NixOS/nixpkgs#270897 . We can update the tests now to find out what else is broken.
Note: the --enable-developer flag got retired so there are no developer builds anymore. Developer features are enabled by the the --developer runtime option. We need to update the tests to do so.

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 a pull request may close this issue.

2 participants