Move LND node communication to background script #244
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Indirect progress towards #46, #42, #78. Should unblock #222.
Description
This refactors the
lib/lnd-http
library to have two API-compatible implementationsLndHttpClient
is now run in thebackground
scriptLndMessageClient
, that has an identical API, but just sends messages to the background script to send.This is an extremely ugly layer of abstraction, not to mention the code is nearly sinful to avoid too much code duplication, but it comes with some major benefits:
Steps to Test
Really the whole app, unfortunately