Integrate KeysInterface to ChannelManager with Arc

Implement KeysInterface for KeysManager util

Add a KeysInterface which provides keys from user

Move ChannelKeys into keysinterface for generate a set of it from
master_seed and change return type to panic on bogus data

Generate Events from ChannelMonitor to indicate spendable ouputs

Extend KeyStorage with delayed_payment_base_key and per_commitment_point
to derive local_delayed private key

Merge pull request #224 from TheBlueMatt/2018-10-221-whitespace

#221 with a few trailing spaces removed

Comments updated for PaymentReceived

Merge pull request #220 from TheBlueMatt/2018-10-oops-xxx

Fix XXXs that slipped into router and handle HTLCFailCHannelUpdates

Run travis-fuzz on Rust 1.29.2 due to 1.30 breaking honggfuzz build

Add PaymentFailReason for fail_htlc_backwards

to indicate fail reason after a PaymentReceived event

Fix XXXs that slipped into router and handle HTLCFailCHannelUpdates

Merge pull request #213 from TheBlueMatt/2018-10-monitor-fail-pause

Add  ChannelManager support for monitor update failure in one place

Add basic sanity testing for channel monitor update failure handle

ChannelManager support for monitor update failure in one place

Add Channel support for monitor-update-failed pausing

Add an Event for sending revoke_and_ack messages out-of-band

Pull out the HTLC forwarding loop into a function

Pull out regenerating RAA/CS in channel_reestablish handling

Move channel monitor updates inside the channel_state lock

This really, really sucks as it defeats almost all of the
cross-channel parallelism we'd intended to have - waiting on a
client to update a watchtower for an unrelated channel to process
any messages is really shitty. We should revisit this with
per-channel locks as a compile-time option post-0.1.

Merge pull request #212 from TheBlueMatt/2018-10-two-updates-disconnect

Fix reconnect message order on remote updates while waiting on RAA

Add test for two-updates-in-flight disconnect channel resumption

Fix reconnect message order on remote updates while waiting on RAA

Add message ordering return value to handling channel_reestablish

Merge pull request #219 from TheBlueMatt/2018-10-157-merge

Partially implement more onion error handling for sent payments

Move HTLCFailChannelUpdate handling out-of-band

While this isn't neccessary for message ordering consistency, this
does mean that we won't end up processing an HTLCFailChannelUpdate
from a update_fail_htlc prior to it being fully committed (where if
the peer disconnects/reconnects it could theoretically give us a
different result, eg if their next-hop reconnected to them).

Partially implement more onion error handling for sent payments

Also refactor out onion error handling into a function instead of
in update_fail_htlc.

Cache the initial htlc_msat that we sent instead of recalculating
it to check validity of the error returned.

Some changes in comments and error messages.

Error handling in decoding onion

Add is_permanent field to ChannelClosed message and add NodeFailure
message

Avoid needless on-chain channel failing for timing-out HTLCs

See new comments in code for more details

Merge pull request #211 from TheBlueMatt/2018-10-misc-cleanups

Miscellaneous Cleanups

Test double-reconnect

Use APIError::ChannelUnavailable a bit more for consistency

Ensure channel is live, not usable for sending fee updates

+ a freebie TODO removal

Use live, not usable for list_usable_channels

Merge pull request #210 from TheBlueMatt/2018-10-funding-disconnected

Fix + test funding tx confirmation while peer is disconnected

Fix + test funding tx confirmation while peer is disconnected

Merge pull request #208 from TheBlueMatt/2018-10-reconnect-fixes

Fix channel_reestablish exchanging in case of lost messages

Add documentation about PaymentSent/Failed event duplication bug

Test reconnecting after lost message(s) during the commitment dance

Fix channel_reestablish exchanging in case of lost messages

This uses the new storage in HTLC state enums to reproduce the
various updates in a CommitmentUpdate group which is obviously
required to re-send a commitment_update after pending unreceived
updates were dropped.

Does not yet handle dropped update_fee updates properly.

Store onion packet in OutboundHTLCState so we can regen update_add

Store per-state HTLC data in the state enum itself (and store more)

Split create_chan_between_nodes into two steps

Merge pull request #206 from TheBlueMatt/2018-10-189-rebase

#189 rebased

Further simplify get_{inbound,outbound}_pending_htlc_stats

Split get_pending_htlc_stats

to get_inbound_pending_htlc_stats and get_outbound_pending_htlc_stats

Check that reserves are met or progress is made on commitment tx gen

Add test for channel_reserve with holding cell HTLCs counted

Add simple test for channel_reserve/max_htlc_value_in_flight_msat

Break up test channel creation to specify values explicitly

Add a ChannelValueStat getter to simplify tests

Document channel reserve behavior a bit better

Fix checking of channel_reserve values

Merge pull request #196 from yuntai/shutdown-apify

Raise APIError from close_channel

add a ChannelUnavailable error

raise APIError from close_channel

Merge pull request #201 from TheBlueMatt/2018-09-channelerror-1

Introduce a Channel-specific Err type and return it in a few places

Use new ChannelError in get_channel_announcement

Use new ChannelError in channel_reestablish handling

Use new ChannelError in HTLC-removal msgs, filling out more handling

Use new ChannelError in funding_locked, filling out more handling

Use new ChannelError type in accept_channel

Introduce a Channel-specific Err type and return it in a few places

This is way simpler than writing out the whole ErrorAction mess and
we can just convert it as appropriate in ChannelManager.

Merge pull request #199 from TheBlueMatt/2018-09-184-fixed-monitor

Fix simple to_local revoked output claim and rebase #184

Merge pull request #200 from TheBlueMatt/2018-09-187-tweaks

#187 rebased + squashed

Macro-out checking a tx validly spends another (and add one more)

Test htlc outputs single tx claim due to timeout case

Test htlc outputs shared tx claim case

Fix and test simple remote-revoked-commitment-output claiming

Add 2-inbound-in-flight update_fee test (that we wouldn't generate)

Annotate test_update_fee to make events more clear

Add test for out-of-order update_fee commitment_signed/RAA

Add test for send_commitment pending non-committed feerate updates

Note that update_fee is likely not gonna be our final API

Add update_fee tests

Expose updating fee on an individual channel

Implement update_fee handling

Macro out one-monitor-added check in network tests (thanks @yuntai)

Originally suggested by Yuntai Kyong <yuntai.kyong@gmail.com> in
some new tests.

Macro out getting payment_preimage/hash in network tests

Merge pull request #197 from TheBlueMatt/master

Fix fuzztarget failures

Avoid double-panic in ChannelManager when tests fail

Handle temporary_channel_id collision in fuzztarget mode

Narrow check_spend_remote_htlc input/output count check.

This fixes a crash found by fuzztester where a 0-output tx causes a
[] panic (though this shouldn't happen in the real-world as
0-output txn should never be able to be mined).

Merge pull request #193 from SWvheerden/Bolt2-specs-errors-of-closing-messages-

Added test for testing sciptpubkey of closing message per bolt 2 spec

Merge pull request #194 from TheBlueMatt/2018-09-doc-cleanup

Make docs look nicer by adding explicit spacing

Added test for testing sciptpubkey of closing message per bolt 2 spec

Make docs look nicer by adding explicit spacing

Merge pull request #191 from TheBlueMatt/2018-09-chanmon-ser-framework

Migrate ChannelMonitor writing/reading to new serialization framework

Bump version to 0.0.5 to get new docs out

Switch to -O1 in dev as we have long-running tests and O1 is cheap

Migrate ChannelMonitor serialization to new ser framework(ish)

Sadly we can't straight up use the new serialization framework as
we have a few different serialization variants, but that's OK, it
looks identical and is just missing the Writeable impl

Move ChannelMonitor deserialization to new ser framework

Simplify DecodeError enum by removing some useless distinctions

Merge pull request #192 from TheBlueMatt/2018-09-docs-docs-docs

Add module and all-pub-things docs and deny missing docs

Add module and all-pub-things docs and deny missing docs

Document ConfirmationTarget a little bit (closes #101)

Document all the fields in Errors

Make message fields pub(crate) and hide a few internal fns

Log peer features in peer_handler (and check for all req bits)

Remove fuzz channel_target.

It has bitrotted a lot and doesn't really make sense given all the
precondition enforcement that is in channelmanager, sadly.