umbrel icon indicating copy to clipboard operation
umbrel copied to clipboard

Enable accepting Atomic Multipath Payment (AMP)

Open aphex3k opened this issue 4 years ago • 5 comments

Since v13.0 lnd supports AMP between paying and receiving lnd nodes.

For umbrel nodes to support receiving amp, they need to be enabled in lnd.conf. This patch allows for both amp and keysend transactions to be received by the node.

See: https://docs.lightning.engineering/lightning-network-tools/lnd/amp

aphex3k avatar Nov 26 '21 02:11 aphex3k

ACK amp enablement would be nice

kevkevinpal avatar May 02 '22 14:05 kevkevinpal

Since LND is already at version 13, could a user simply enable Atomic Multi-path Payments (AMP) by updating their lnd.conf file?

  • SSH into your node
  • cd ~/umbrel/lnd
  • sudo nano lnd.conf
  • Add "accept-amp=1" in your lnd.conf file
  • Save and Exit
  • Reboot or Restart LND

Should that work?

jasonpearce avatar May 07 '22 13:05 jasonpearce

Is there anything else needed for this to be merged? It's a tiny change, but it would solve a lot of problems related to big payments. I know that we can modify the config on our own, but then we'll have to remember to reapply the change after each Umbrel update.

rafftyl avatar Jul 18 '22 19:07 rafftyl

I know that we can modify the config on our own, but then we'll have to remember to reapply the change after each Umbrel update.

@rafftyl changes you make to lnd.conf are persisted. If you add accept-amp=1 it will stay there after app updates and Umbrel updates.

Is there anything else needed for this to be merged?

We'll need to make the change over in the getumbrel/umbrel-apps repo since the Lightning app has migrated there. We have some updates planned for the Lightning app shortly so we can add this as part of the next update.

lukechilds avatar Jul 28 '22 12:07 lukechilds

@lukechilds

changes you make to lnd.conf are persisted.

Oh, that's cool. Thanks for the info.

rafftyl avatar Jul 28 '22 12:07 rafftyl

Closing as this was addressed in this commit: https://github.com/getumbrel/umbrel-apps/commit/7d7b37f02e2b7ebdc70b7ad4460151f410fe4fd6

nevets963 avatar Sep 10 '22 21:09 nevets963

amp is now enabled by default in the latest version of the Lightning app, thanks for your suggestion @aphex3k!

lukechilds avatar Sep 14 '22 05:09 lukechilds