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.

Merge pull request #29 from TheBlueMatt/main

Update to latest upstream rust-lightning, bumping deps

Update Cargo.lock with updated deps

Update to latest upstream rust-lightning, bumping deps

Merge pull request #26 from TheBlueMatt/main

Display our local balance in addition to the available-for-send balance

Merge pull request #24 from jkczyz/2021-07-background-processor

Explicitly stop BackgroundProcessor

Display our local balance in addition to the available-for-send balance

Merge pull request #25 from valentinewallace/2021-06-keysend

Keysend

Keysend support

Don't force announced channel preference

Merge pull request #22 from TheBlueMatt/main

Allow multiple publicly-announced addresses and avoid funding panic

Explicitly stop BackgroundProcessor

An upcoming change will stop the BackgroundProcessor's thread upon drop.
Store the processor in a variable and explicitly stop it. Otherwise, the
thread would be stopped immediately after it is started.

Merge pull request #23 from valentinewallace/2021-07-0.0.99-bump

Update to RL upstream 0.0.99

Update to RL upstream 0.0.99

Do not panic if we fail to fund a channel

I saw this where a bitcoind was hung connecting a block, and while
we were waiting the peer refused the channel via an error message.
When bitcoind eventually returned, we paniced as the channel was
now gone.

Allow multiple publicly-announced addresses

Merge pull request #21 from TheBlueMatt/main

Correct Feerate Calc and Tweak Funding Feerate

Use a reasonable target fee when opening channels

Bitcoin Core defaults to conservative fee estimation, which often
results in significant overtargeting on weekends. We also disable
RBF, here, though, with a comment describing how clients should
handle funding RBF.

Correct feerate units

Lightning, unlike every other Bitcoin-related protocol or API, uses
satoshis-per-1000-weight, instead of
satoshis-per-1000-virtual-bytes. This requires a division by 4 to
calculate the correct fees when passing Bitcoin Core fees into LDK.

Merge pull request #20 from TheBlueMatt/main

Include bad-txns-inputs-missingorspent in allowed tx-broadcast errs

Include bad-txns-inputs-missingorspent in allowed tx-broadcast errs

I saw this with Bitcoin Core 0.21.2, I guess "Inputs missing or
spent" was renamed.

Merge pull request #14 from TheBlueMatt/main

Persist network graph

Persist network graph

Merge pull request #17 from jkczyz/2021-06-guide-updates

Updates to align with Using LDK guide

Simplify loop in connect_peer_if_necessary

Remove outdated comment on payment secret

Simplify extracting fields from invoice

Initialize UserConfig in one statement

Merge pull request #16 from valentinewallace/update-to-0.98

Update to RL upstream 0.0.98

Update to RL upstream 0.0.98

Merge pull request #12 from jkczyz/2021-05-event-handler

Process events with BackgroundProcessor

f - Rename steps to match Build a Node guide

Process events with BackgroundProcessor

Merge pull request #13 from valentinewallace/fix-broadcast-tx-panic

Don't panic if broadcast_transaction fails due to duplicate broadcasts

Don't panic if broadcast_transaction fails due to duplicate broadcasts

or from broadcasting too early.

Merge pull request #11 from valentinewallace/expose-channel-publicness

Indicate whether a channel is public in `listchannels`

Merge pull request #10 from valentinewallace/fix-log

Fix badly formatted log

Fix broken 'open public channels' feature

Indicate whether chan is public in listchannels

Update `help` to include new commands and clean up parameters

Fix badly formatted log

Otherwise there's random \t\t's in the log

Merge pull request #7 from TheBlueMatt/main

Small tweaks to logging and printing + node_announcement

Fix on-startup peer connection due to missing await.

Print more clear information about channel state in listchannels

User feedback found `channel_can_send_payments` confusing on its
own, and now that we have an upstream `is_confirmed` field, print
that.

Further, it is useful to have an explicit txid field, even if if is
basically duplicative with channel_id.

Broadcast node_announcements regularly.

Update payment error messages to better match reality

Merge pull request #8 from jkczyz/2021-05-git-dependencies

Use dependencies from git when building locally

Use dependencies from git when building locally

Specifying a version number for dependencies prevents building against
recent changes upstream. However, either `path` or `git` can be given to
be used locally while `version` is still used when publishing to a
registry like crates.io. For details, see:

https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#multiple-locations

Merge pull request #6 from TheBlueMatt/main

Make connections non-blocking

Switch to ToSocketAddrs to support connecting to hostnames not IPs

Make panic message for port-already-bound more human readable

Make everything unreadable so rustfmt is happy

Connect to peers with tokio TcpStreams isntead of blocking tokio

Fix inbound connections by using tokio net instead of std

Merge pull request #5 from TheBlueMatt/main

Trivial Usability Fixes

Adopt rustfmt's completely braindead expected formatting

Print the local Node ID on startup

Bump to release 0.0.14

Check that the node we connect to matches the user-provided chain

Make an initial dummy RPC call to check that the connection works

This provides a much better panic message if the RPC password is
incorrect.

Set panic = abort as we don't want to keep running if we panic.

Fix minor indentation issue

Merge pull request #4 from valentinewallace/sample-v0

Sample v0

Update to latest RL + fix step numbers + nodeinfo + listpeers commands

Adapt to the lightning_invoice crate moving to Rust-Lightning

Check payment secret before succeeding payment

Split payments hashmap into inbound and outbound

To prevent clashes between payment hashes

Remove HTLCDirection struct

We're going to split payment storage into two maps, one for
inbound and one for outbound.

SatoshiAmount -> MillisatAmount and PaymentInfo struct

Update to latest upstream RL

Move startup to separate from `main` for better errors

Include min_final_cltv when generating invoices

Read routehints from invoices when sending a payment

Remove unnecessary todo

Update to RL master and fix PendingHTLCsForwardable

Post-async refactor cleanup

Initial refactor to add tokio::main and be fully async

working before async refactor

add rustfmt + CI + other fixups

Forward HTLCs after a timeout and a few other cleanups

Add readme, fix license and a few other cleanups

Next draft. Features: connectpeer, listchannels, sendpayment, autoreconnect to chan peers on startup

Initial node

Fee estimator WIP

Initial commit

Merge pull request #3 from jkczyz/2021-03-dual-license

Apache 2.0 / MIT dual license

Apache 2.0 / MIT dual license

Initial commit