Refactor check spendable outputs macros into one

In consequence, harden spendable outputs tests

Fix vocabulary abuse

Merge pull request #254 from TheBlueMatt/2018-11-channelerror

Move a ton of Channel functions to ChannelError from HandleError

Fix two compiler warnings introduced in a51dbb4a4de014238ab67ba507c

Move a ton of Channel functions to ChannelError from HandleError

This is a big patch, but its all very mechanical, everything here
should be pretty obvious, and it all has to happen at once due to a
few common utility functions all having the same return type.

Note that this exposes a race in channel closure where we may
access a channel via some non-peer-specific mechanism like
forwarding an HTLC or sending a payment during the time between
the channel gave us a Close error and expected us to never call it
again and the time we actually removed it from the channel_state
set outside of the internal_* handler.

Create simple ChannelMonitor-specific Err type

Merge pull request #231 from philipr-za/philip-204-check-commitment-transaction-fee

Check funder can afford commitment transaction fee when receiving update_fee

Merge pull request #230 from ariard/handle_sizeable_push_msat

Handle sizeable push msat (fix #195) + handle two first per_commitment_point + keys interface tests

Add test_static_output_closing_tx

Add test_dynamic_spendable_outputs_local_htlc_*

Cover both local HTLC-Timeout/Success case

Add key_storage selection in ChannelMonitor insert_combine

Based on commitment_number

Add test_static_spendable_outputs_justice_tx_revoked_htlc*

Cover both HTLC-Timeout/Success cases

Add test_static_spendable_outputs_justice_tx_revoked_commitment_tx

Add test_claim_on_remote_sizeable_push_msat

Contrary to sizeable push_msat on local
commitment tx, the output go to a P2WPKH

Add check_spend_closing_transaction ChannelMonitor

Aims to send back closing output descriptor to user wallet

Add test_static_spendable_outputs_preimage_tx

Aims to covered both keysinterace preimage tx case and
detection of second remote commitment tx

Split DynamicDescriptor between *P2WSH and *P2WKH

Merge pull request #253 from TheBlueMatt/2018-11-misc-tweaks

Misc Tweaks

Split provide_their_next_commitment_point from provide_secret

We needed it to be able to track remote_per_commitment_point
after channel opening and funds locking

Add test_claim_sizeable_push_msat

Aims to cover both claiming of sizeable_push_msat and
spendable output generation for to_local output

Add TxOut in DynamicOutput

Track local_commitment_tx at funding_created

Goal to claim sizeable push_msat and in event of
local commitment tx being broadcast without htlcs

Implement spendable output to_local output on local commitment tx

Merge pull request #249 from stevenroose/bitcoin-0.15

Update to rust-bitcoin v0.15

Panic if funding transaction has non-witness inputs

Clarify get_update_*_htlc docs and tweak one case slightly

Expand test_funding_peer_disconnect somewhat by being non-symmetric

I thought I found a bug in one-side-funded-first reconnect, but
seems I can't reproduce it here. Either way worth improving the
test coverage.

Check affordability of new fee during update_fee call #204

Update to rust-bitcoin v0.15

Merge pull request #248 from TheBlueMatt/2018-11-monitor-test-split

Split up channel_monitor_network_test a little bit

Merge pull request #247 from TheBlueMatt/2018-11-202-redux

Handle-initial_routing_sync-requests-from-peers-in-their-Init-messages

Split up channel_monitor_network_test a little bit

Handle-initial_routing_sync-requests-from-peers-in-their-Init-messages

Add test justice tx on revoked htlc_success

Extend channel_monitor_network_test to cover
all check_spend_remote_htlc cases

Merge pull request #242 from yuntai/201811-malformed

Fail channel when BADONION is not set

fail channel when BADONION is not set.

Merge pull request #241 from TheBlueMatt/2018-10-peer-free-panic

Fix pre-noise outbound peer disconnect panic found by fuzzer

Merge pull request #233 from TheBlueMatt/2018-10-shutdown-updates

Shutdown Updates

Test the various shutdown handling updates

Rebroadcast shutdown on channel_reestablish (and reprocess them)

Delay closing_signed until update_fee exchanges complete

See https://github.com/lightningnetwork/lightning-rfc/issues/499
for a bit more about the ambiguity we're addressing here.

Also drop holding cell update_fee the same way we drop holding
cell update_add_htlcs when sending shutdown, resolving a bug.

Fail incoming HTLCs sent after we start shutdown, not the chan

Avoid double-panic in full_stack_target to assist debugging

Fix pre-noise outbound peer disconnect panic found by fuzzer

If we make an outbound connection to a peer who we are already
connected to, and the outbound connection fails
pre-noise-completion, we will remove the original peer connection
from our node_id_to_descriptor map.

The fuzzer managed to find this by crashing in Channel's assertions
that we don't do a get_channel_reestablish() when the Channel isn't
already marked disconnected.

Add logging for (dis)connect in peer_handler/channelmanager

Send closing_signed when appropriate as pending HTLCs clear

Always send Shutdown resposnes to Shutdown messages

We always require that any changes to Channel state be committed
immediately (within the same lock) so we should never have
uncommitted changes which would prevent us from sending a Shutdown
response.

Refactor out first-closing-sent creation

Use ChannelError in shutdown handling, clean up, impl err actions

Add warnings reminding users to use SegWit funding inputs

Merge pull request #235 from TheBlueMatt/2018-10-229-redux

Bug fix using same seed sequence for channel key geneartion

Bug fix using same seed sequence for channel key geneartion

Merge pull request #234 from TheBlueMatt/2018-10-169-redux

169 Tweaked

Added config interface to allow users to specify channel limits

Require at least 1k sat for outbound channels and meet BTC's dust

Use non-funder's funding block wait instead of max with ours

This is both required by the protocol and also makes sense - if
we're the funder we don't mind accepting payment on the channel
after one confirmation because we assume we won't double-spend
ourselves.

Merge pull request #232 from TheBlueMatt/master

Fix a full_stack_target crash

[full_stack_target] don't allow double-confirming txn

Give ManyChannelMonitor a logger and trace add_update events

Merge pull request #226 from TheBlueMatt/2018-10-0.0.6

Bump version to 0.0.6

Merge pull request #223 from TheBlueMatt/2018-10-chanmanager-serialize

Implement and document Channel/ChannelManager (de)serialization

Merge pull request #222 from TheBlueMatt/2018-10-pre-commit-disconnect

Fix and test update_add_htlc but disconnect pre-commitment_signed

Merge pull request #217 from TheBlueMatt/2018-10-msg-resp-overhaul

Send ChannelManager messages out-of-band to ensure ordered delivery

Merge pull request #225 from TheBlueMatt/2018-10-214-redo

Keys Interface Simplification

Bump version to 0.0.6

Add very basic test of ChannelManager serialization round-trip

fix broken state comparison (thanks @yuntai!)

Implement and document Channel/ChannelManager (de)serialization

Fix + test disconnect/reconnect prior to FundingLocked

Fix and test update_add_htlc but disconnect pre-commitment_signed

Avoid reentrancy of send_data from PeerHandler::read_bytes.

This greatly simplifies clients of PeerHandler, and because almost
all response messages have already been moved to process_events
this doesn't change much effeciency-wise.

Integrate destination_script from KeysInterface in ChannelManager/Channel

Drop channel_monitor_claim_key from ChannelKeys

Store+expose bits of Channel[Monitor] to figure out local state

Send channel_reestablish out-of-band to ensure ordered deliver

Integrate shutdown_pubkey from KeysInterface in ChannelManager/Channel

Drop channel_close_key from ChannelKeys

Track ChannelMonitor-watched-outpoints (+ remove now-uesless Mutex)

Respond to channel_reestablish out-of-band for ordered delivery

Integrate KeysInterface to ChannelManager with Arc

Track last_block_hash in ChannelMonitor and expose it on deser

Also make block_connected take a &mut self to ensure serialized
state will always be self-consistent.

Send RAA/CS messages out-of-band to ensure ordered delivery

Implement KeysInterface for KeysManager util

Redo ChannelMonitor deserialization to avoid read_to_end()

This slightly changes the serialization format, but we're still
early enough that that's OK.

Send shutdown/closing_signed msgs out-of-band for ordered delivery

impl some additional (de)serializers, including a u48 wrapper type

Send announcement_signatures msgs out-of-band for ordered delivery

impl Readable/Writable for Route

Send funding_signed messages out-of-band to ensure ordered delivery

Add a BIG lock to ChannelManager

During normal operation we should never need to take this, so we
use a RwLock that allows normal parallelism until we want to
serialize out our ChannelManager, at which point we can take the
write-mode lock.

Send AcceptChannel responses out-of-band to ensure ordered delivery

Give ChannelMonitor a logger via new ReadableArgs trait

Split Event, move MessageSendEvent push() inside channel_state lock

Add a second Readable trait for state args, clean macros slightly

Store [u8; 32]s instead of SharedSecrets (for deserialization)

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