Handle duplicate payment_hash send_payment()s

We don't bother doing anything smart, we just return an Err in
case we have a pending outbound HTLC with the payment_hash provided
for the send.

This doesn't resolve the TODO as users can still send duplicative
payments that have the same payment_hash as was already sent,
though the docs are updated and hopefully users do so on their own.

Merge pull request #93 from TheBlueMatt/2018-07-announce-bug

Properly calculate Channel::announce_publicly

Remove excess mut warned on in 1.22

Properly calculate Channel::announce_publicly

For some reason we were only setting "announce_publicly" when
Channel::new_from_req had announce_publicly set to true and the
open_channel message had the relevant flag set. However, this
resulted in us rejecting peers for sending unsolicited
announcement_signatures messages, despite them having indicated,
and us having accepted, their announce-bit-set open_channel.

Merge pull request #90 from TheBlueMatt/2018-07-percent-bump

Bump README percent usable

Fix some tab-space errors in #84

ACHIEVEMENT UNLOCKED

Merge pull request #84 from savil/sort_outputs

[RFC][Tx Sort] Implement sorting of inputs

Merge pull request #92 from savil/editorconfig-fix

fix editorconfig so it applies to every file

adding BIP69 test-cases

fix editorconfig so it applies to every file

Merge pull request #89 from TheBlueMatt/2018-07-route-fuzz-fix-1

Fix fee addition overflow in get_route

Fix fee addition overflow in get_route

Merge pull request #87 from savil/editorconfig

add .editorconfig to ensure we use tabs, not spaces

Merge pull request #88 from TheBlueMatt/2018-07-new-fuzzing

Add Router Fuzz Target

Only abort channel_target fuzz test on Err if ErrorAction says to

Add router_target to fuzzing which fuzzes router msgs + get_route

Catch fee overflow panic in get_route

Update NodeAnnouncement addr deserialization to check addr len.

This more aggressively checks the message contents are correct
before returning WrongLength so existing fuzz setup has an easier
time.

Ensure all msg deserialization can provide extra bytes, rename err

add .editorconfig to ensure we use tabs, not spaces

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

[RFC][Tx Sort] Implement sorting of inputs

Follows BIP69: https://github.com/bitcoin/bips/blob/master/bip-0069.mediawiki

1. Implements sorting of transactoin inputs.
- BIP says to use "reversed byte-order" for the `prev_hash`. I interpreted this as: little-endian.
- TODO need to add tests

2. Re: improve sorting of TxOut's script_pubkey to use lexicographic ordering, and not length.
From the test-cases i've included it seems that the current code already does lexicographic ordering (and not length based). Am i missing something?

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.