cdecker / lightning-integration

Lightning Integration Testing Framework
Other
74 stars 31 forks source link

Update LND to support version from September 2018 #38

Closed ysangkok closed 5 years ago

ysangkok commented 5 years ago

This breaks Eclair because it doesn't yet support differing URLs for ZMQ block and tx notifications. But I can repair this once https://github.com/ACINQ/eclair/issues/733 is solved.

cdecker commented 5 years ago

I think most of the changes in here have already been addressed? Could you confirm that things are working as expected?

ysangkok commented 5 years ago

Almost works, see #42

ysangkok commented 5 years ago

No, actually it doesn't seem to work for me, which makes sense since there is no unlocking and noencryptwallet is not passed. Did it work for you after commit 089f2d6afc70f5742e7c2ee47e20d8d54dba078f?