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

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

Print next and previous nodes for PaymentForwarded

Update to LDK 0.0.107

Merge pull request #57 from meryacine/main

Add a help message for the `keysend` command

Add a help message for the `keysend` command

Merge pull request #56 from TheBlueMatt/main

Stop connecting outbound on shutdown in addition to stopping listen

Stop connecting outbound on shutdown in addition to stopping listen

Now that we persist the network graph on shutdown, there is a
nontrivial amount of time spent between when we start shutdown and
when we complete. Thus, and to be somewhat more pedantic as a
sample implementation, we should really stop making new outbound
connections once shutdown starts, and not only focus on new inbound
connections.

Merge pull request #51 from jkczyz/2022-03-ldk-106

Update to LDK 0.0.106

Update Cargo.lock

Update to LDK 0.0.106

Merge pull request #55 from TheBlueMatt/2022-03-node-alias

Merge pull request #54 from TheBlueMatt/main

Avoid panic if chan IDs passed to {force,}closechannel isn't 32 bytes

Print our peers' aliases in `listchannels`

Allow `txn-mempool-conflict` errors when broadcasting transactions

This can be seen, among other times, when a channel is
cooperatively- (or counterparty-)closed and we restart, causing us
to attempt to broadcast our local commitment tx, which conflicts
with the original closing transaction.

Add a bounds check before accessing a user-provided PK string

Fixes #53.

Merge pull request #52 from TheBlueMatt/main

Cleanly shut down PeerManager by disconnecting before we quit

Correct balance printing in cli

The `outbound_capacity_msat + unspendable_punishment_reserve`
calculation is fine, as long as the channel's current balance is
above the reserve value. If it is not, we'll spuriously report a
balance higher than we actually have.

This corrects that be havior by using the new(er) `balance_msat`
field instead.

Cleanly shut down PeerManager by disconnecting before we quit

This ensures we don't keep processing peer messages long after we
stopped the backgroundprocessor (which results in our
`ChannelManager` on disk being stale compared to monitors).

Merge pull request #46 from TheBlueMatt/2022-03-0.0.105

Update to LDK 0.0.105

Use new `ProbabilisticScorer` instead of the deprecated `Scorer`

Update Cargo.lock

Update to 0.0.105 upstream API

Merge pull request #43 from TheBlueMatt/2021-12-0.0.104

LDK 0.0.104

Switch keysend command to using InvoicePayer to get retries

Expose the fee paid on a payment in stdout

Drop GOSSIP-level logging at compile-time

Update Cargo.lock

Update to LDK 0.0.104

Merge pull request #42 from dongcarl/2021-11-add-signet

Add Signet support

Add Signet support

Fix signmessage offset calculation

Auto-reconnect peers every second

Include total local balance across all channels in `nodeinfo`

Print CLI prompt at top of loop instead of before each `continue`

Merge pull request #40 from jkczyz/2021-11-release-103

Update sample for 0.0.103

Persist channel scores

Use InvoicePayer in send_payment

Update dependencies to LDK 0.0.103

Merge pull request #39 from TheBlueMatt/main

Update dependencies to LDK 0.0.102

Swap time for chrono and log subsecond precision for times

The `time` crate appears to be basically undocumented (or, at
least, I couldn't find any documentation). `chrono` isn't super
well-documented, but at least its passable.

Add lightning message-signing support

Update dependencies to LDK 0.0.102

Merge pull request #37 from radixrat/main

use rsplitn for rpc info extraction in case passwords have @

use rsplitn for rpc info extraction in case passwords have @

remove swap, fix rustfmt issue

remove return

Merge pull request #36 from lightningdevkit/moneyball-patch-1

update installation instructions to work even if ssh keys aren't setup

update installation instructions to work even if ssh keys aren't setup

Merge pull request #35 from TheBlueMatt/2021-09-0.0.101

Update to rust-lightning 0.0.101 API and use release, not git deps

Use an explicit handle when spawning RPC calls

This resolves a panic like the following, which is caused when a
non-Tokio thread tries to broadcast a transaction, for example
when a transaction is broadcasted indirectly by the background
processor.

```
thread '<unnamed>' panicked at 'there is no reactor running, must be called from the context of a Tokio 1.x runtime', src/bitcoind_client.rs:253:9
```

Update to rust-lightning 0.0.101 API and use release, not git deps

Merge pull request #34 from lightningdevkit/2021-08-fix-33

Correctly parse args without port but with net

Merge pull request #28 from sr-gi/fix-sendrawtransaction

Makes send_raw_transaction return TxidHex instead of RawTx

Correctly parse args without port but with net

Fixes #33.

Makes send_raw_transaction return Txid instead of RawTx

'bitcoind_client::send_raw_transaction' was returning 'RawTx' which, while syntactically correct (it's a wrapper around String), made no sense conceptually.

Since rust-bitcoin/rust-lightning@f65d05c, `Txid` can be used instead.

Bumps lightning dependencies

'rust-lightning' needs to be bumped to v0.0.100 rev d523b6e so 'TryInto::<Txid>' can be found in 'lightning-block-sync'. Bumping it any further will change 'PeerManager's API due to https://github.com/rust-bitcoin/rust-lightning/commit/45853b3a956f83569dff1010a43a33c57487698a

Merge pull request #32 from lightningdevkit/2021-08-skip-change-check

Drop unused change output calculation

Drop unused change output calculation

We don't need this at all anymore since LDK automatically scans
for the correct output index these days.

Fixes #31.

Merge pull request #30 from lightningdevkit/2021-08-more-failure-codes

Allow "rejecting replacement" tx broadcast error

Allow "rejecting replacement" tx broadcast error

We occasionally will attempt to broadcast transactions which conflicting
with mempool transactions which are marked RBF. If we don't have a
higher fee, we'll get a "rejecting replacement" error, which should not
cause a panic.