add same docker support
modify sample docker compose file
include lndk-cli in the docker image
fix docker compose file
update dockerfile to use github and use slimmer image
lower grace period
use better lnd paths
add some lnd config
The `cargo-dist` tool has been renamed to `dist` to reflect the fact
that it is becoming a general purpose tool to distribute binaries.
We update `cargo-dist` to `dist` automatically by running `dist init`.
Note the generation of the new custom configuration file which will
be used by `dist` going forward.
this commit aims to fix lndk hanging when lnd is killed
Processes did not stop and would faild to send or receive onion messages
even if lnd came back online. Quick solution implemented in this commit
is to close up the server when receiving those errors.
LNDK is already using a datadir to store TLS certificates.
In this PR we default LNDK config and data to ~/.lndk.
Datadir is moved inside the default dir ~/.lndk/data. Then,
we expect lndk.conf file to be iether in root of the project,
or the default LNDK config folder or --config flag.
Precedence is flag > root > default dir.
In certain deployment scenarios, we may not want (or be able) to write
to the default ~/.lndk data directory. For example, the home directory
may be mounted as read-only, or mounted using ephemeral storage.
This change adds a new `data_dir` configuration option to specify where
LNDK should store data. If this option isn't specified, then falls back
to the default ~/.lndk.
In the README, example variable names for `lndk.conf` were incorrect.
They need to be snake_case rather than kebab-case.
Specifically:
`cert-path` --> `cert_path`
`macaroon-path` --> `macaroon_path`
This commit updates the README with the proper variable names.
Additionally, this commit changes the wording for clarity by specifying
LNDK "features" rather than "options".
We had removed this expiry delta because payments to more recent versions of
LDK and CLN were working without it. But without it, some payments to other
versions fail.
In earlier versions of ldk-sample we couldn't make a payment without this
additional delta. But CLN doesn't like this additional delta and it's no longer
necessary for payments to ldk-sample.
