Test transaction watch registration in channelmonitor tests

Ensure the funding transaction is registered to be monitored

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

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)

Merge pull request #28 from TheBlueMatt/master

fuzztarget sha -> XOR, crates secp256k1

fuzztarget sha -> XOR, crates secp256k1

Merge pull request #26 from TheBlueMatt/master

Small style tweaks for new message targets for my own style preferences

Update msg_target_template to note auto-generation

Fix indentation in fuzz_target utils

Make travis check that msg_targets match the template

Merge pull request #24 from yuntai/master

fill decode/encode routines for many messages with fuzz targets

add fuzz target for messages
[[bin]] section for each new target in Cargo.toml
glob chanbes in travis-fuzz.sh

add script to generate msg test target

Update dependency on secp256k1 pointing to https://github.com/rust-bitcoin/rust-secp256k1

filling unimplemented!() for many messages' encoding/decoding routines

add .gitignore

Merge pull request #25 from TheBlueMatt/master

Fix channel_target crash fp

Make channel_target return as channelmanager removes on shutdown

Some with_capacity's

Merge pull request #23 from TheBlueMatt/master

Fix repo link in Cargo

Fix repo link in Cargo

Merge pull request #22 from TheBlueMatt/master

Bump version for API changes.

Bump version for API changes.

Merge pull request #21 from TheBlueMatt/master

Begin handling errors/tx broadcasts robustly

Fix local route update broadcasts, annouce channel closures

Rewrite channelmonitor framework and implement a bunch of it

Fix some channel remove channelmanager consistency bugs

HTLC Failure message handling

Move ChannelKeys creation into channelmanager

Merge pull request #20 from TheBlueMatt/master

Rewrite parts of channel, cleanup a bunch of stuff

Provide fallback for crypto's fixed_time_eq on non-x86/arm targets

Minor cleanups

Fix commitment transaction number/per_commitment_point tracking

Refactor channelmanager network tests significantly

impl more message encodings

Completely rewrite channel HTLC tracking and processing

It obviously was nonsensical prior, though its now way more
complicated...there's refactoring work to be done here, but it
should at least kinda sorta work now.

Prep HTLCFailReason for Channel holding it until revoke_and_ack

Use libbitcoinconsensus to check tx validity in some tests

Test that channel monitors get updated appropriately