Merge pull request #117 from TheBlueMatt/main

Sweep Fixes

Dont block tokio reactor waiting on user input

When we moved the CLI input reading into a `tokio::select`, we
ended up blocking the tokio reactor by reading from stdin in a
standard tokio future. Instead, we need to let tokio know that
we're running a largely-blocking task.

Correct directory path for spendables sweep

Merge pull request #114 from TheBlueMatt/2023-08-last-ditch-write

Attempt a last-ditch ChannelManager persistence if the BP exits

Attempt a last-ditch ChannelManager persistence if the BP exits

If the BP exits because it failed to write our ChannelManager to
disk, we'll probably lose some channels on startup. However,
there's still some chance for us to write the `ChannelManager` in a
retry, which we should do before we exit.

Merge pull request #116 from wpaulino/anchors

Add support for opening anchor outputs channels

Merge pull request #112 from TheBlueMatt/main

Trim whitespace when signing messages

Add anchors flag to openchannel CLI command

Handle Event::BumpTransaction via BumpTxEventHandler

Implement bump_transaction::WalletSource for BitcoindClient

`BitcoindClient` will be used as the view into the onchain wallet from
which we'll source UTXOs.

Manually accept inbound channels

Since we plan to add support for opening anchor outputs channels, we'll
need to support manually accepting inbound channels first to do so.

Trim whitespace when signing messages

Fixes https://github.com/lightningdevkit/rust-lightning/issues/2188

Merge pull request #109 from valentinewallace/2023-07-detect-bgproc-exit

Detect if the background processor exits early and shutdown if so

Merge pull request #110 from tnull/2023-08-no-chrono-default-features

Don't use `chrono` default features

The `chrono` crate has a dependency on an old version of `time` which
has a potential segfault. We therefore disable the default features.

Detect if the background processor exits early and shutdown if so

Merge pull request #107 from tnull/2023-07-upgrade-to-0.0.116

Upgrade to LDK 0.0.116

Upgrade to LDK 0.0.116

Merge pull request #104 from zoedberg/persist_payments

persist payments info to disk

persist payments info to disk

Merge pull request #106 from tnull/2023-07-fix-github-actions

Update CI to remove deprecated actions

Merge pull request #108 from domZippilli/2023-07-relax-peer-reconnect-tick-behavior

Switch missed tick behavior to delay instead of default burst

switch missed tick behavior to delay instead of default burst

Update CI to remove deprecated actions

Merge pull request #103 from TheBlueMatt/main

Make SpendableOutput claims more robust

Try to sweep `SpendableOutputs` regularly, and in bulk

Rather than trying to sweep `SpendableOutputs` every time we see
one, try to sweep them in bulk once an hour. Then, try to sweep
them repeatedly every hour forever, even after they're claimed.

Fixes #102.

Announce node_announcemnet even without listen addresses

Even if we don't have any listen addresses, it's still useful to
broadcast a node_announcement to get our node_features out there.

Here we do this and also improve the timing of our
node_announcement updates to be less spammy.

Merge pull request #100 from TheBlueMatt/main

Update to LDK 0.0.115

Drop windows-latest stable rustc build as it now fails

Needs a backport of the fixes in [1] to ntapi 0.3.

[1] https://github.com/MSxDOS/ntapi/issues/15

Update Cargo.lock

Log ChannelPending events

Print the ultimate payment failure reason on PaymentFailed events

Switch to using the async background processor

This drops a full thread from the sample node and better integrates
LDK into the tokio runtime. While we're at it we also push the
`Event` being processed by move into the event handler, which
doesn't matter much here but makes further modification in the
event handling somewhat simpler.

Update to LDK 0.0.115

Drop `futures` dependency with `tokio::select`

Manually poll'ing a future every 10ms rather than selecting across
it and a timer is less effecient, and certainly not worth taking a
dependency to do.

Merge pull request #99 from TheBlueMatt/main

Don't panic on tx broadcast failures

Don't panic on tx broadcast failures

In general LDK will often broadcast transactions which aren't
actually able to get into the mempool. That's generally fine,
though could indicate a significant issue. Thus, we previously had
a panic in place for it to ensure developers are notified and we
can investigate.

However, now that people are trying to actually *use* ldk-sample,
we should just remove the panic and print something with more
details.

Pass a logger through into `BitcoindClient`

Merge pull request #97 from valentinewallace/2023-02-0.0.114

Update to LDK version 0.0.114

Update to LDK v0.0.114

Merge pull request #96 from TheBlueMatt/main

Its 2023, its time to support v6

Its 2023, its time to support v6

Merge pull request #95 from freddiekrugerrand/01-22-startupargs

Prompt user to provide at least 2 arguments rather than 3

Prompt user to provide at least 2 arguments rather than 3

env::args returns the name of the binary, which isn't something
the end user would think about. So while we require at least 3
arguments, it makes sense to only ask for 2.

Merge pull request #94 from TheBlueMatt/main

Restore MSRV to 1.48

Restore MSRV to 1.48

This reverts 810054ead2e022a771e902c26327313597947b96 and reverts
some `Cargo.lock` changes in fbb1f231ef0e1e8362d353e5d4e3aaecba42a

Merge pull request #93 from tnull/2023-01-update-to-v0.0.113

Update to LDK release 0.0.113

Raise MSRV to 1.49.0, following tokio

Update for LDK release 0.0.113

Merge pull request #89 from alecchendev/parse-env-followup

Followup: Parse .cookie, .env, or environment variables for RPC auth details

Merge pull request #85 from ch1ru/main

Add `disconnectpeer` subcommand

Change order of where to find rpc auth

Fix testnet datadir path

Merge pull request #87 from alecchendev/parse-env

Parse .cookie, .env, or environment variables for RPC auth details

Parse .cookie, .env, or environment variables for RPC auth details

Merge pull request #78 from MaxFangX/fix-chanman-race-panic

Fix channel manager race panic

Remove `chain_tip`'s unneeded `Option`

Fix channel manager race panic

The race:
- (1) If a channel manager is starting for the first time, it gets its
  initial block hash by calling BlockSource::get_blockchain_info() and
  constructing a BestBlock out of that.
- (2) During the initial chain sync, if chain_tip.is_none() then the
  best chain_tip is retrieved using
  lightning_block_sync::init::validate_best_block_header() which
  similarly uses BlockSource::get_best_block()
- If blocks were mined between (1) and (2), the channel manager’s block
  hash will be at a lower height than that of the block hash contained
  inside the chain_tip fed to the SpvClient. Suppose that this results
  in the channel manager being at block 1 and the SpvClient being at
  block 2. Once block 3 comes in and the SpvClient detects it via
  poll_best_tip(), the SpvClient computes the ChainDifference between
  its saved chain_tip value and block 3, concluding that it just needs
  to connect block 3. However, when Listen::filtered_block_connected is
  called using the data in block 3, the channel manager panics with
  “Blocks must be connected in chain-order - the connected header must
  build on the last connected header” - because block 3’s prev block
  hash is block 2’s block hash, rather than block 1’s block hash which
  the channel manager expects

This commit fixes it by removing the fresh channel manager's query to
the block source, instead deriving its best block from the validated
block header using a new `.to_best_block()` method.

Merge pull request #86 from MaxFangX/update-lock

Update Cargo.lock to reflect new package name

Update Cargo.lock to reflect new package name

Add `disconnectpeer` subcommand

Disconnect peer with `disconnectpeer <pubkey>`.
Checks all channels are closed before disconnecting.

Merge pull request #82 from TheBlueMatt/main

Update to LDK 0.0.112

Update Cargo.lock

Provide an MSRV of 1.48

Update to LDK 0.0.112

Merge pull request #84 from tnull/2022-10-update-package-name

Update package name to `ldk-sample`

So far the name was `ldk-tutorial-node`, we now update this to align
with the repository.

Merge pull request #83 from ch1ru/main

Semantically sort subheadings

Semantically sort subheadings

Merge pull request #77 from tnull/2022-10-catch-ctrlc

Catch Ctrl-C and enable Ctrl-D

Add CI jobs for `windows-latest` and `macos-latest`

Ignore Ctrl-C and enable Ctrl-D

We ignore Ctrl-C/SIGINT via a dummy signal handler and enable the use of
Ctrl-D. For convenience, we introduce "exit" and "quit" cli commands.

Merge pull request #79 from MaxFangX/fix-blocksource-impl

impl `BlockSource` on `BitcoindClient` instead of `&BitcoindClient`

impl `BlockSource` on `T`, instead of `&T`

Merge pull request #72 from MaxFangX/make-send-sync

Make `ldk-sample` `Send` + `Sync`

Make `ldk-sample` `Send` + `Sync`

This makes it possible to run ldk-sample in a tokio task, e.g.

```rust
pub async fn main() {
    tokio::spawn(async move {
        start_ldk().await;
    })
        .await
        .expect("LDK panicked")
}
```

Merge pull request #70 from valentinewallace/2022-09-ldk-0.0.111

Update to LDK 0.0.111

Merge pull request #71 from tnull/2022-09-improve-hex-utils

Small improvements of `hex_utils`

Small improvements of `hex_utils`

Update to LDK 0.0.111 and add command for sending an empty onion message

Merge pull request #65 from standardsats/main

Update README.md

Merge pull request #68 from tnull/2022-09-05-dont-double-check-addr

Remove redundant `is_none()` check

Remove redundant `is_none()` check

We don't need to check the same iterator twice. And, as at this point
pubkey will always be `Some(..)`, we don't need to check it either.
We therefore can just remove the first part of the condition.

Merge pull request #67 from catenocrypt/main

Add --public option to openchannel command help

Add --public option to openchannel command help

Update README.md

Merge pull request #66 from tnull/2022-07-update-to-0.0.110

Update to LDK v0.0.110

Update to v0.0.110

Merge pull request #64 from TheBlueMatt/main

Correct logging on forwarded HTLCs

Instead of "forwarded from A from B" it should read "forwarded
from A to B".

Merge pull request #62 from TheBlueMatt/main

Update to LDK 0.0.109

Update Cargo.lock with tokio 1.14.1, matching our MSRV

Update code and Cargo.toml to LDK 0.0.109

Merge pull request #59 from jkczyz/2022-06-ldk-108

Update Cargo.lock

Update to LDK 0.0.108

Merge pull request #58 from jkczyz/2022-05-ldk-107

Update to LDK 0.0.107

Update Cargo.lock

Use a 10-second timeout for InvoicePayer

Avoid cloning Arcs when handling each event