940053d000
htlc: move enum side crom daemon/channel.h to daemon/htlc.h
lightningd wants htlcs, but not the old struct channel.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
7d68554787
type_to_string: add secp256k1_ecdsa_signature.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
25a37fafae
type_to_string: add privkey.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
d97e3489a0
daemon/htlc_state: split off from daemon/htlc.
We want to use struct htlc inside lightningd, and we need the state
bits.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
c6997f15c7
bitcoin/preimage: struct preimage.
We had a hack for 'struct rval' in protobuf_convert.h; make an
explicit header and put it in bitcoin/preimage.h. It's not really
bitcoin-specific, but it's better than having bitcoin/script depend on
an external header.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
279f216208
gen_peer_wire_csv: update to latest spec.
In particular, 860990fa0afb55f839e882a5e9abe8abe6ccb981 reordered
channel_announcement and c93bf5cf8c48eab1b028e85214cb35feeeffcbb3
reordered the update_fail_malformed_htlc message.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
f9eea9041e
handshake: update for BOLT 8 latest revision.
This now matches commit fc4846a27bf420c591ba0b024cc34972816913c7,
and fixes "make check-source".
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
a02f1e10c4
make check: run all the tests.
We can't run them in parallel, but we can at least have 'make check'
run them all.
Developers should be running "make check-source && make check".
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
05cf3160d4
ccan: Added ccan/intmap
8 years ago
999472d99a
bitcoin: fix script number encoding.
Encodings are signed: we may need 5 bytes to encode giant u32s.
Reported-by: Fabrice Drouin <fabrice.drouin@acinq.fr>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
8522a5ea64
struct bitcoin_tx: remove explicit lengths, use tal_len()/tal_count()
They're always tal objects, so we can simply ask tal for the length,
simplifying the API.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
ad82d29379
ccan: update to get tal_len(NULL)
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
8159c4458a
bitcoin/script.h: remove struct bitcoin_signature
Technically this incudes the sighash flags, but we only handle SIGHASH_ALL
anyway.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
1edce4878d
bitcoin/signature.h: remove struct signature.
It's a wrapper around secp256k1_ecdsa_signature, so use that directly.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
e3d966c102
type_to_string: add channel_id
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
a25e2816e9
type_to_string: add secp256k1_pubkey
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
66175c7361
Merge pull request #115 from ElementsProject/pytest-pathfix
pytest: don't assume paths of valgrind and bitcoind.
8 years ago
1575181ef5
pytest: don't assume paths of valgrind and bitcoind.
In particular, my bitcoind is in /usr/local/bin
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
1c49af7486
Merge pull request #114 from cdecker/pytest-new
pytest: Add support for both legacy and new daemons
8 years ago
1948062250
pytest: Using python-bitcoinlib instead of python-bitcoinrpc
The latter is unmaintained and has a number of problems.
8 years ago
180c96776b
pytest: Split the python integration tests into new and legacy
8 years ago
5f61b3a272
pytest: Separating new lightningd and legacy lightningd RPC client
We intend to ultimately no longer use the legacy `daemon/lightningd`
and instead use `lightningd/lightningd`, so I grouped the new RPC and
the legacy RPC and implemented stubs for the new daemon.
8 years ago
091c2fc8f5
log: Flushing logs on each new entry
This is to speed up the python testing framework and should not have a
big impact on performance.
8 years ago
7cc5e2fc7a
pytest: Added pytest target to Makefile
8 years ago
cf1c03f4d9
pytest: Implemented initial python integration tests.
8 years ago
41a5e74db9
pytest: Implement utilities for python unittests
8 years ago
cd9bb9b014
contrib: Implemented python RPC client
8 years ago
3f79a0e117
log: Flushing logs on every line
This is needed for the new testing framework since we wait for
messages to be printed on stdout. Buffering delays this
arbitrarily. Flushing so often should not have much of a performance
impact.
8 years ago
2fb2e757ae
doc: add lightning-waitinvoice man page.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
69079e51a9
doc: move Makefile parts into doc/
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
df59cef3af
doc: rename waitinvoice manpage to waitanyinvoice.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
d6ccf90063
jsonrpc: Renamed `awaitpayment` -> `waitinvoice` -> `waitanyinvoice`
As suggested by Rusty
8 years ago
1a9bfe7be4
jsonrpc: Added `awaitpayment` method
`awaitinvoice` can be used to wait on a specific invoice to be
completed. If the invoice was previously paid, then the command
returns immediately, otherwise it'll block until the invoice is
paid. This complements `waitinvoice` which uses a highwatermark and
waits for the next invoice. I found waitinvoice a bit hard to use
since it doesn't allow waiting for a specific invoice to be completed,
just the next in the insertion order.
8 years ago
ba83430b5f
trivial: Removing unused constant
8 years ago
cae283087d
sphinx: Committing the onion packet to the payment-hash
The sphinx onion packet now commits to the HTLC payment-hash it is
associated with. This prevents replay attacks with the same onion.
8 years ago
91b17d45d8
sphinx: Removing last vestiges of the end-to-end payload
So far this was simply set to a zero-length end-to-end payload. We
don't have any plans of re-adding it for the moment, so let's get rid
of the unused code.
8 years ago
285b8b4698
sphinx: Use libsecp256k1 to generate shared secrets
So far we did it on our own, but since the spec specifies that we use
the libsecp256k1 version anyway, we can remove our own implementation.
8 years ago
679dec3e6a
sphinx: Actually use the full pubkey in ECDH key generation
The spec says that we use the libsecp256k1 style ECDH, which uses the
full compressed pubkey from the scalar multiplication which is then
hashed. This is in contrast to the btcsuite implementation which was
only using the hashed X-coordinate.
8 years ago
3d20cf559a
gitignore: Ignoring some of the test-binaries.
8 years ago
6bf3c30a10
lightningd/Makefile: fix check-source
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
b41d71da30
Makefile: Making sure header vars are populated correctly
The problem with wire headers not being generated in time before stuff
depended on it turns out to be related with inclusion order of
sub-makefiles. The inclusions must preceed the use of
LIGHTNINGD_HEADERS since they append to that variable.
8 years ago
91209d4d66
ccan: forgotten test files after last update.
Also add ccan/config.h to .gitignore.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
33748bf119
lightningd/Makefile: combine all headers (wire/gen was missing)
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
73d07ce441
libsodium: use our local submodule.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
1c6e28eaac
libsodium: add as submodule.
We need 1.0.9+ for the IETF chacha variants; Ubuntu xenial only ships
with 1.0.8.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
3af081aded
Makefile: make all lightningd objects depend on all lightningd headers.
Christian reported that we weren't generating some gen_ files.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
1e34e5344d
lightningd/lightningd: add getpeers command.
This has an optional log level if you want to see logs for the peers.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
dc8b174e3a
lightningd/lightningd: maintain a per-peer log.
At the moment we just log each condition change.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
83b156517f
lightningd/lightningd: maintain "condition" for each peer.
This lets us return better messages to the connect json command on failure.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
7aaffda779
lightningd/lightningd: finish connect command once gossip started.
This is after the INIT message is received, so we know there are no
incompatible features.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
8 years ago
Rusty Russell
Christian Decker