Use the neutrino switcher with lncm instead of getumbrel

runcitadel / compose

GNU Affero General Public License v3.0

6 stars 2 forks source link

Use the neutrino switcher with lncm instead of getumbrel #3

Open nolim1t opened 3 years ago

nolim1t commented 3 years ago

Reference: https://github.com/runcitadel/compose/blob/8382d47227c8ce095bf4d0dc3860cd9af2634106/docker-compose.yml#L148

LNCM version: https://github.com/lncm/docker-lnd-neutrino-switch

Umbrel version: https://github.com/getumbrel/docker-lnd-neutrino-switch

Reason(s):

Umbrel version is really out of date
Better to keep dev decentralized and in one place, ideally where the development is done in
The only reason why umbrel forked it was to change some things in the upstream which they could easily could have made a pull request. I've fixed whatever they needed in the upstream and more.

nolim1t commented 3 years ago

In addition if you want to use the lncm lnd.. theres a valid reason for it that we are building other versions of it. The experimental tag in LND enables better routing while LND is running in neutrino mode

This can possibly be switched out using neutrino-switcher too

; Makes routing faster but have to use images built with experimental tag
[Routing]
routing.assumechanvalid=1

AaronDewes commented 3 years ago

Umbrel has feeurl set by default (which is required on neutrino mainnet) and removes it after sync (https://github.com/lncm/docker-lnd-neutrino-switch/commit/905bb0d503e133abed33057ad8d27b9621d66203) It seems like this is still missing upstream, can you please add that too? I'll switch to your version then.

nolim1t commented 3 years ago

Done 1.0.5

AaronDewes commented 3 years ago

Thanks! I'll update later, my internet has issues today