Ensure all msg deserialization can provide extra bytes, rename err

Merge pull request #86 from TheBlueMatt/2018-07-event-fixes

Fix one bug on event handling, clean up docs

Make claim_funds_INTERNAL private

Do not pass DisconnectPeer events upstream to the client

PendingHTLCsForwardable really should just be upstreamed.

This is just a comment/docs/function rename change

Merge pull request #85 from TheBlueMatt/2018-07-fees

Convert fee API to per_kw instead of per_vb

Convert fee API to per_kw instead of per_vb

This (a) cuts down a bit on some conversions, reducing chances for
bugsand (b) provides greater accuracy for clients.

Merge pull request #82 from TheBlueMatt/2018-07-send-initial

Initial fixes for payment sending

Fix HTLC exclusion balance calculation criteria

Half-ass some CLTV fixes for real channel creation

Set associated_data to payment_hash

Merge pull request #83 from TheBlueMatt/2018-07-direct-route

Short-circuit router for an outbound specified direct channel

Short-circuit router for an outbound specified direct channel

Merge pull request #79 from TheBlueMatt/2018-07-chanmon-oom

Fix OOM crash in ChannelMonitor::deserialize found by AFL

Fix OOM crash in ChannelMonitor::deserialize found by AFL

Merge pull request #78 from TheBlueMatt/2018-07-43-rebased

Add DisconnectPeer event

Add DisconnectPeer events

Add test for DisconnectPeer event
Update DisconnectPeer with optional ErrorMessage
Manage error for funding_transaction_generated
Add disconnect_socket to SocketDescriptor trait

Implement ErrorMessage msg and ErrorAction::SendErrorMessage + fuzz test

Merge pull request #77 from TheBlueMatt/2018-07-route-tweaks

Add arg to get_route to specify our local channels explicitly

Add arg to get_route to specify our local channels explicitly

(and a new getter in ChannelManager to get the thing to pass in)

Skip always iterating over all nodes in get_route

Merge pull request #76 from TheBlueMatt/2018-07-75-dangling-spaces

Remove dangling spaces from #75

Remove dangling spaces from #75

Merge pull request #75 from TheBlueMatt/2018-07-68-rebased

[msgs::ChannelReestablish] implement encoding and decoding

Merge pull request #74 from TheBlueMatt/2018-07-shutdown-event

Move shutdown sending to an event

[msgs::ChannelReestablish] implement encoding and decoding

Summary:

Implementing these, taking inspiration from the surrounding code since i'm a rust n00b.

Test Plan:
```
cargo build
cargo test channel_reestablish
```
output:
```
running 3 tests
test ln::msgs::tests::encoding_channel_reestablish_with_secret ... ok
test ln::msgs::tests::decode_channel_reestablish_bad_length ... ok
test ln::msgs::tests::encoding_channel_reestablish_no_secret ... ok

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 14 filtered out
```

Merge pull request #69 from savil/gen-target-fix

[fuzz] remove \L in sed regex for filenames in gen_target.sh

Move shutdown sending to an event

Merge pull request #73 from TheBlueMatt/2018-07-no-uint

Migrate all Uint256s used for channel_ids to [u8; 32]

Migrate all Uint256s used for channel_ids to [u8; 32]

Merge pull request #71 from TheBlueMatt/2018-07-watch-funding

 Ensure the funding transaction is registered to be monitored

Fix failure sending FundingLocked for non-announced channels

Test transaction watch registration in channelmonitor tests

Ensure the funding transaction is registered to be monitored

Merge pull request #70 from TheBlueMatt/master

Clarify get_peer_node_ids docs a bit

Clarify get_peer_node_ids docs a bit

Merge pull request #67 from yuntai/issue_56

#56 Expose list of pubkeys of connected peers from PeerHandler and addresses-by-pubkey from Router

add two methods for the issue #56
peer_handler::get_peer_node_ids() get connected peers' node_id
router::get_addresses() to get network addresses by node_id

[fuzz] remove \L in sed regex for filenames in gen_target.sh

Summary:
on `master` branch, if i ran `gen_target.sh` i would get the following output:
https://gist.github.com/savil/2b3114eaf34c3ef8499b65005e374841

Without it, I get the files generated that are already checked-in (as expected).

Looking at the sed docs (https://www.gnu.org/software/sed/manual/html_node/Regular-Expressions.html) i don't see an explanation for what `\L` may do in whatever regex sed follows. Does this look correct? cc @yuntai

Merge pull request #65 from TheBlueMatt/master

Bump version for API overhauls, update deps and docs

Merge pull request #53 from savil/rename-error-msg-to-action

[HandleError] rename msg field to action

Bump version for API overhauls, update deps and docs

Merge pull request #47 from ariard/block_disconnected_close_chan

Implement channel closing for block_disconnected on ChainListener + test

update TODO message for renaming "msg" field of HandleError struct

[Error] rename msg field to action

this was a TODO and also briefly discussed in https://github.com/rust-bitcoin/rust-lightning/pull/43#pullrequestreview-135649551

I'm not fully sure how to remove the `Option`, and make it completely required. Would love suggestions. So, have omitted that for now. Plus, better to make smaller, incremental changes.

Test Plan:
`cargo build`
`cargo test`

Make some more things pub(crate) as they don't need exposed now

Merge pull request #64 from TheBlueMatt/master

 Expose the script_pubkey being monitored in chain watch (fixes #44)

Implement channel closing for block_disconnected on ChainListener + test

Remove in-repo TODOs (they're now tracked in GitHub)

Expose the script_pubkey being monitored in chain watch (fixes #44)

Merge pull request #52 from TheBlueMatt/master

Fix various spec bugs, can now open channels with real nodes!

Fix various spec bugs, can now open channels with real nodes!

 * commitment transaction number, as used in locktime/sequence
   fields is actually different from commitment transaction number,
   as used for revocation state. This is confusing and never stated
   in the spec, so we have to do the conversion.
 * max_htlc_value_in_flight is never constrained in the spec, but
   we were requiring it be <= channel size. Instead just clamp the
   values the peer sends us when storing.
 * channel_id calculation was incorrect, we now do some crazy
   conversion hops, which we shouldn't, but will need to change our
   types to fix.
 * Our channel_reserve_satoshis value was too low, just change the
   constant and leave the TODO to figure out what it really should
   be for now.

Merge pull request #49 from ariard/bad_failure_codes

Erroneous failure codes, replace PERM|7 by UPDATE|7

Merge pull request #51 from TheBlueMatt/master

Shorten channelmonitor_deserialize_target's name for AFL

Shorten channelmonitor_deserialize_target's name for AFL

Merge pull request #50 from TheBlueMatt/master

Fix channelmonitor fuzz test failure

Fix channelmonitor fuzz test failure

Merge pull request #48 from TheBlueMatt/2018-07-monitor-serialize

 Implement channelmonitor (de)serialization

Erroneous failure codes, replace PERM|7 by UPDATE|7

Test channelmonitor serialization roundtrip doesn't mutate state

Implement channelmonitor (de)serialization (fixes #45)

Clarify ChannelMonitor remote commitment tracking and fix a race

Move channelmonitor-update functions to pub(ln) from pub

Update TODOs in README

Merge pull request #46 from ariard/useless_todo_channelmanager

TODO already cleaned by #16

TODO already cleaned by #16

Merge pull request #41 from TheBlueMatt/master

Move open_channel message handling into an event

Move open_channel message handling into an event

This simplifies client usage cause peer_handler doesn't have an
interface to shove arbitrary messages in.

Merge pull request #39 from TheBlueMatt/master

Add warning about aut-close()-fd-races for peer_handler impls

Add warning about aut-close()-fd-races for peer_handler impls

Merge pull request #38 from jeandudey/max-funding-satoshis

Add `MAX_FUNDING_SATOSHIS` constant.

Add `MAX_FUNDING_SATOSHIS` constant.

This constants defines the maximum value of `open_channel.funding_satoshis`,
currently it's defined to be 2^24 according to the BOLT #2 specification.
Also a test was added to check that the constant is never over 2,1x10^15
(maximum satoshis in bitcoin) if modified.

Signed-off-by: Jean Pierre Dudey <jeandudey@hotmail.com>

Merge pull request #37 from TheBlueMatt/master

Fix additional 1.22 rustc warning

Merge pull request #36 from TheBlueMatt/2018-08-33-cleanups

Cleanups after #33, plus one unrelated bugfix spotted in #33 review

Fix additional 1.22 rustc warning

Add more docs to transaction::OutPoint

Cleanups after #33, plus one unrelated bugfix spotted in #33 review

Merge pull request #35 from TheBlueMatt/2018-06-post-32

Some cleanups after #32

Merge pull request #33 from jeandudey/funding_txo

Use the `TxOutRef` type for the `ChannelMonitor`'s funding_txo field.

Greatly simplify channelmonitor pruning tests, and use real funcs

Instead of hooking into internal data structures and modifying
those, call the actual functions Channel will call and then check
that the preimages map is correct.

Also avoid pruning preimages for previous local tx in ChannelMonitor

Fix chan_utils indentation issue

A few minor test cleanups and remove new warnings from #32

Skip some a needless temporary vec and clean indentation after #32

Merge pull request #32 from ariard/prune_payment_preimages

Add pruning of preimages no longer needed + tests

Add the `OutPoint` type for the `ChannelMonitor`'s funding_txo field.

This structure replaces the (Sha256dHash, u16) tuple that was being used
for the funding output.

Signed-off-by: Jean Pierre Dudey <jeandudey@hotmail.com>

Add pruning of preimages no longer needed + tests

Merge pull request #34 from TheBlueMatt/master

Fix Pong response attempt-to-encode-oversize-msg panic

Fix Pong response attempt-to-encode-oversize-msg panic

Merge pull request #30 from TheBlueMatt/master

Working initial routing sync from lnd node

Fix message targets to not have an internal loop

Add ping/pong fuzz targets

Send initial_routing_sync-filled Init messages to the first 5 peers

Support responding to Ping messages in peer_handler

Work around lnd sending invalid messages

Support ignoring some errors, deserialize empty flags types

Merge pull request #29 from TheBlueMatt/2018-06-api-quirks

Some API Quirks

Make PeerHandleError Error impl public

Remove bitcoincorerpcchain, as it belongs in a separate crate

Provide Send + Sync on message handlers (just changes trait)