eugene
f82653cb06
fuzz: change return values to 1 rather than 0 or -1
...
This allows gofuzz to store the mutating input as coverage if it reaches
any new coverage, even if it didn't make it to the end of the test.
2020-10-13 15:17:57 -04:00
Johan T. Halseth
d26001a69a
Merge pull request #4688 from carlaKC/4686-paymenttimeout
...
lncli: allow configurable timeout in send payment
2020-10-13 13:06:47 +02:00
carla
0a76f87bd0
lncli: allow configurable timeout in send payment
2020-10-12 13:27:06 +02:00
Oliver Gugger
f98c74319c
Merge pull request #4617 from guggero/no-txindex-itest
...
itest: add bitcoind test without txindex enabled
2020-10-09 15:51:20 +02:00
Oliver Gugger
4dcdfd98bb
travis: add bitcoind test without txindex
2020-10-09 13:35:06 +02:00
Oliver Gugger
3e3618ae9a
lntest: add build flag for disabling txindex on bitcoind
...
We create a new build flag for running the bitcoind tests without the
txindex enabled. We don't want this to be the default so we use a
negated build flag.
2020-10-09 13:35:04 +02:00
Oliver Gugger
b7fc7c3dae
lntest: use extraArgs, fix linter issues
2020-10-09 13:35:02 +02:00
Oliver Gugger
43c2031fa8
lntest: extract common bitcoind code into own file
...
To prepare for running multiple tests on bitcoind with different options
each time, we extract the common code into its own file.
2020-10-09 13:35:00 +02:00
Oliver Gugger
902d7dd812
Merge pull request #4643 from guggero/make-fuzz
...
Add fuzzing commands to Makefile
2020-10-09 11:03:30 +02:00
Oliver Gugger
41d8d56354
gitignore: ignore fuzzing artifacts
2020-10-09 09:53:56 +02:00
Oliver Gugger
1a1a9a6744
mod: add auto-added go mod changes
...
When running the make DEPGET target for the go-fuzz binaries, this entry
is auto added, even though we don't have a direct reference to it in our
code. Not sure why this is needed in the first place.
2020-10-09 09:53:56 +02:00
Oliver Gugger
5189cb799a
make: add fuzz-build and fuzz commands
2020-10-09 09:53:55 +02:00
Oliver Gugger
916b75fd4e
Merge pull request #4680 from guggero/travis-bitcoind-speedup
...
Travis+GitHub: update bitcoind to 0.20.1, extract bitcoind binary from docker image to speed up download
2020-10-09 09:43:52 +02:00
Conner Fromknecht
33fe44f4ca
Merge pull request #4681 from guggero/itest-whitelist
...
itest: add error to whitelist
2020-10-08 17:19:57 -07:00
Oliver Gugger
e8253479d5
Travis+GitHub: update to bitcoind 0.20.1, remove cache
...
With this commit we use the same version of bitcoind in all tests and
also remove the caches for the binary as docker has its own cache.
2020-10-08 21:30:11 +02:00
Oliver Gugger
785e797ef8
scripts: install bitcoind binary from docker image
...
Because the bitcoind mirror is extremely slow, we spend at least 2 to 3
minutes of each bitcoind related test on just downloading the binary. We
can achieve the same result by just pulling the docker image and
extracting the binary from that.
2020-10-08 21:29:51 +02:00
Olaoluwa Osuntokun
7e1d825834
Merge pull request #4554 from Roasbeef/go-1-15
...
build: build and run CI against Go 1.15
2020-10-08 12:18:22 -07:00
Oliver Gugger
eb74011d5a
itest: add error to whitelist
2020-10-08 20:39:11 +02:00
Johan T. Halseth
22d7bb138c
Merge pull request #3131 from halseth/reliable-payments-switch-gc
...
[reliable payments] switch result store clean-up
2020-10-08 15:43:34 +02:00
Olaoluwa Osuntokun
6493a43c9d
Merge pull request #4668 from Empact/patch-2
...
[autopilot] change default heuristic to top_centrality
2020-10-07 16:48:12 -07:00
Olaoluwa Osuntokun
0f90bb638d
build: build and run CI against Go 1.5
2020-10-06 15:30:50 -07:00
Olaoluwa Osuntokun
c7e948bb5a
Merge pull request #4673 from guggero/psbt-nested-segwit-fix
...
PSBT funding: don't include scriptSig field for nested SegWit inputs
2020-10-06 15:22:37 -07:00
Oliver Gugger
843efc324b
walletrpc: serialize TXID not outpoint
...
The txid_str field of a locked UTXO's outpoint shouldn't contain the
output index but only the reverse order hex serialized transaction ID.
2020-10-06 10:59:51 +02:00
Oliver Gugger
8e9f37b0fc
lncli+docs: fix outpoint serialization in JSON output
...
This commit fixes the JSON output of the outpoint field of locked UTXOs
in the response of the PSBT fund command.
2020-10-06 10:59:50 +02:00
Oliver Gugger
0c0d114d18
mod: bump btcwallet dep to fix scriptSig in PSBT bug
...
To fix a bug where the scriptSig field was mistakenly set on nested
SegWit inputs of a PSBT, we bump the btcwallet dependency that contains
said fix.
2020-10-06 10:59:46 +02:00
Johan T. Halseth
90a59fe70f
router: call CleanStore on startup
2020-10-06 10:46:04 +02:00
Johan T. Halseth
b071278f07
htlcswitch: add cleanStore method
...
That let us clean up handed off payment results, as they will never be
queried again.
2020-10-06 10:46:03 +02:00
Johan T. Halseth
d1a118388d
payment_control: remove unused Attempts field
2020-10-06 10:41:25 +02:00
Johan T. Halseth
46225e2921
Merge pull request #4666 from halseth/autopilot-mis-totalfunds
...
[autopilot] Use local channel balance in allocation calculation
2020-10-05 14:56:20 +02:00
Johan T. Halseth
dbeafe8832
autopilot: rename to Capacity to Balance, define ChannelInfo()
...
We rename the field Capacity on local channels to Balance, define a new
method ChannelInfo on the manager config that let us query the database
for our latest channels state. Using this we will use the current local
balance instead of the channel capacity when doing allocation
calculations.
2020-10-05 12:24:07 +02:00
Johan T. Halseth
d40cf6b592
autopilot: split channel definition into LocalChannel/ChannelEdge
...
Since non-local channels won't have a balance field, we split the
definitions in anticipation of adding one.
2020-10-05 12:24:07 +02:00
Johan T. Halseth
3e1755ee99
autopilot: remove unimpleneted SpliceIn/Out methods
2020-10-05 12:16:50 +02:00
Johan T. Halseth
392d6ccc9f
autopilot: remove unused channel field FundedAmt
2020-10-05 12:16:50 +02:00
Johan T. Halseth
b6c161bcfc
Merge pull request #4671 from Roasbeef/health-check-logging
...
healthcheck: bump logging for failed healthchecks to info
2020-10-05 10:59:09 +02:00
Olaoluwa Osuntokun
35a2dbcabb
healthcheck: bump logging for failed healthchecks to info
...
In this commit, we bump up the logging for failed healthchecks from
`debug` to `info`. This should help us get to the bottom of the current
set of reported false positives that are causing `lnd` nodes to
erroneously shutdown.
2020-10-04 16:31:51 -07:00
Oliver Gugger
f8e3b41d10
Merge pull request #4389 from guggero/psbt-signing
...
walletrpc: add new PSBT creation+signing RPCs
2020-10-03 11:47:55 +02:00
Oliver Gugger
84dfed3fe2
docs: update PSBT documentation
2020-10-03 10:34:57 +02:00
Oliver Gugger
eb280fd248
itest: use new PSBT functions for funding flow
...
Now that we have all functions that we need to complete the whole
PSBT channel funding flow, we change the itest to use Dave's wallet
to fund the channel from Carol to Dave through a PSBT.
2020-10-03 10:34:56 +02:00
Oliver Gugger
f114fb3c8d
itest: use require library
...
We rewrite the test to use the require library to make it a
bit more condensed.
2020-10-03 10:34:56 +02:00
Oliver Gugger
c206d062d5
lncli: add new PSBT wallet commands
2020-10-03 10:34:55 +02:00
Oliver Gugger
9a063355e2
walletrpc: disallow locking with internal ID
...
The internal lock ID that the wallet kit subserver uses to lock inputs
for itself shouldn't be allowed to be used when locking inputs manually
over the RPC.
2020-10-03 10:34:55 +02:00
Oliver Gugger
6229609be7
lnrpc+walletrpc: add RPCs for new PSBT methods
2020-10-03 10:34:53 +02:00
Oliver Gugger
f947576f33
lnwallet+mock: add new PSBT methods
2020-10-03 10:34:39 +02:00
Oliver Gugger
9f5f98e736
btcwallet: use new base wallet methods
2020-10-03 10:34:39 +02:00
Oliver Gugger
936a83858b
chanfunding: use util functions from psbt package
2020-10-03 10:34:39 +02:00
Oliver Gugger
8e0a162b8b
mod: update to latest btcwallet version
2020-10-03 10:34:34 +02:00
Ben Woosley
465c2d17f4
autopilot: Change default heuristic to top_centrality
2020-10-02 20:13:54 -07:00
Ben Woosley
da805c022e
Fix top_centrality in sample-lnd.conf
...
Input:
> autopilot.heuristic={topk_centrality:1}
Result:
> strconv.ParseFloat: parsing "1}": invalid syntax
Input:
> autopilot.heuristic=topk_centrality:1
Result:
> heuristic topk_centrality not available. Available heuristics are: [ 'preferential' 'externalscore' 'top_centrality' ]
2020-10-02 15:09:52 -07:00
Conner Fromknecht
652ac7fa3f
Merge pull request #4652 from MrManPew/patch-3
...
lnd: show debug level at startup
2020-10-02 13:16:23 -07:00
Eugene
10a84f2c75
Merge pull request #4649 from Crypt-iQ/fmgr_abstract
...
multi: add fmgr.Manager interface, change usage in peer to use Manager
2020-10-02 08:21:15 -04:00