Oliver Gugger
8eebd4feba
mod+docs: bump btcwallet to latest version
2022-12-05 09:33:10 +01:00
Oliver Gugger
91c0a19807
Merge pull request #6913 from bottlepay/relax-failure-len-check
...
htlcswitch: relax failure message length check
2022-12-02 16:00:21 +01:00
Joost Jager
4c8ea29336
lnwire: allow longer failure messages
...
This fixes an incompatibility where lnd enforces a strict 256 byte
failure message, where as the spec sets this only as the recommended
length.
2022-12-02 14:27:59 +01:00
Oliver Gugger
e614d806e5
Merge pull request #7204 from YusukeShimizu/fix-liveliness-typo
...
Unified valiable name and comment
2022-12-02 13:10:55 +01:00
Joost Jager
5ff5838d74
lnwire: add extra opaque data to FailIncorrectDetails
2022-12-02 09:28:03 +01:00
Joost Jager
5f4465b14f
link: ensure minimum failure reason length
...
This commit modifies the link behavior so that every failure
reason that we pass back is length-compliant (>=256 bytes).
2022-12-02 09:28:02 +01:00
Joost Jager
9730bc1ca0
lnwire: verify failure message length
...
Adds extra checks to make sure the failure message
is well-formed.
2022-12-02 09:04:59 +01:00
Joost Jager
e9440a24a2
htlcswitch/test: more realistic mock encryption
...
This mock is used in the switch test TestUpdateFailMalformedHTLCErrorConversion.
But because the mock isn't very realistic, it doesn't detect problems
in the handling of malformed failures in the link.
2022-12-02 09:04:59 +01:00
Oliver Gugger
eca82d986c
Merge pull request #7207 from guggero/gossip-flag-fix
...
lncfg: fix gossip.pinned-syncers flag description
2022-11-28 14:01:11 +01:00
Oliver Gugger
00eda60df5
docs: add release notes
2022-11-28 09:58:32 +01:00
Oliver Gugger
542bbbbc30
lncfg: fix gossip.pinned-syncers flag description
...
The gossip.pinned-syncers flag is declared as a string slice which means
it should not be a comma separated list of elements but the flag should
be specified multiple times instead.
2022-11-28 09:55:18 +01:00
bruwbird
274938ed45
fix typo liveliness
2022-11-25 21:52:00 +09:00
Oliver Gugger
71fc52dfe3
Merge pull request #7193 from guggero/taproot-default-change
...
lnwallet/btcwallet: use taproot addrs for change outputs from SendCoins
2022-11-22 17:11:44 +01:00
Oliver Gugger
e9fb8f691f
Merge pull request #7194 from ffranr/fix_dockerfile_comment
...
docker: correct comment concerning lnd build/install
2022-11-22 16:05:31 +01:00
Olaoluwa Osuntokun
aa20bc14d7
docs/release-notes: add entry for 0.15.5
...
Adds an entry for the change addr bug fix, and a missing entry for the
funding locked sleep fix.
2022-11-22 13:44:55 +01:00
Olaoluwa Osuntokun
d75c86f3dd
lnwallet: skip dual funding test as not exposed on p2p layer
...
The internal messages both sides exchange haven't been updated to send
all the information needed to compute a tapscript sighash.
2022-11-22 13:44:54 +01:00
Olaoluwa Osuntokun
7090ff22d7
lnwallet: assert that P2TR addrs are used for change outputs
2022-11-22 13:44:54 +01:00
Olaoluwa Osuntokun
51f131b019
build: update to latest version of btcutil+btcwallet
2022-11-22 13:44:54 +01:00
ffranr
8dfb2023a9
docker: correct comment concerning lnd build/install
2022-11-22 11:54:24 +00:00
Oliver Gugger
ca501d920e
Merge pull request #7188 from guggero/go-func-vars
...
Don't access loop variables in goroutines directly
2022-11-22 11:06:42 +01:00
Oliver Gugger
ee753923e4
docs: add release notes
2022-11-21 13:54:24 +01:00
Oliver Gugger
55b53555e9
multi: improve readability of goroutine defers
...
This commit fixes the readability of some of the defer calls in
goroutines by making sure the defer stands out properly.
2022-11-21 13:54:24 +01:00
Oliver Gugger
ad8e25cbc9
multi: don't access loop variables in goroutines
...
This commit makes sure that no loop variables or other temporary
variables are accessed directly in a goroutine but are instead passed
into the goroutine through a parameter. This makes sure a copy of the
value is put on the stack and is not changed while the outside loop
continues.
2022-11-21 13:54:23 +01:00
Oliver Gugger
e7170ef819
Merge pull request #7135 from st4rgut24/revise-addinvoice-help-text
...
revise help text for addinvoice lncli command
2022-11-18 18:43:58 +01:00
Oliver Gugger
87ca48f4b4
Merge pull request #7173 from yyforyongyu/update-linter
...
golangci: update linter to fix linter bug
2022-11-18 18:25:05 +01:00
yyforyongyu
84fd911b47
multi: fix make lint
...
Fixes new lint errors caught by the latest version.
2022-11-18 20:48:23 +08:00
Oliver Gugger
43ff90a0ff
Merge pull request #7172 from guggero/bitcoind-taproot-itest
...
itest: temporary fix for taproot test failing on bitcoind
2022-11-18 12:42:57 +01:00
yyforyongyu
39b972afe0
golangci: update version to v1.50.1
...
This commit updates the tool `golangci-lint` to the latest version that
has fixed a bug in the linter `paralleltest`. Also `go mod tidy` is run
inside `tool`, and `nosnakecase` is disabled as it's deprecated.
2022-11-18 19:42:38 +08:00
Oliver Gugger
649b9671fb
docs: add release notes
2022-11-18 12:42:05 +01:00
Oliver Gugger
f1124a0ff9
itest: fix taproot itest timeout issue
2022-11-18 11:10:12 +01:00
Oliver Gugger
67e2e382bc
Merge pull request #6822 from yyforyongyu/3-new-itest
...
itest: continued itest refactor and fix - II
2022-11-18 10:27:26 +01:00
yyforyongyu
edeb8c7444
docs: update release notes
2022-11-17 21:06:13 +08:00
yyforyongyu
aafe4c38fc
lntest: fix flake in channel backup related tests
...
This commit fixes the channel backup tests by decreasing the recovery
window used from 1000 to 100 to speed up the startup of node Dave.
2022-11-17 21:06:13 +08:00
yyforyongyu
3d7aae42b3
lntest: increase node start timeout
2022-11-17 21:06:13 +08:00
yyforyongyu
03dae039a7
lntemp: remove balanceCount from node state
...
Since channels are always closed when tests finish, there's no need to
track them in node's state.
2022-11-17 21:06:13 +08:00
yyforyongyu
b68ac6acdc
golangci: update gomnd config
2022-11-17 21:06:13 +08:00
yyforyongyu
60da823396
itest: fix make lint
2022-11-17 21:06:12 +08:00
yyforyongyu
4f9f2f7901
github: update action versions
2022-11-17 21:06:12 +08:00
yyforyongyu
2fe7413f0e
itest: increase test timeout to 120m
...
We now update our test timeout from 60m to 120m because most of the
tests would now take more than one hour to finish.
2022-11-17 21:06:12 +08:00
yyforyongyu
666f877710
lntemp: use h.NoError inside HarnessRPC
2022-11-17 21:06:12 +08:00
yyforyongyu
966e037d53
itest: refactor testFailingChannel
2022-11-17 21:06:12 +08:00
yyforyongyu
008ef964fc
lntemp+itest: refactor testChannelForceClosure
2022-11-17 21:06:12 +08:00
yyforyongyu
61364c0d9d
lntemp+itest: refactor testCommitmentTransactionDeadline
2022-11-17 21:06:12 +08:00
yyforyongyu
f7f259b92a
itest: refactor testChannelUnsettledBalance
2022-11-17 21:06:11 +08:00
yyforyongyu
8f472bf063
itest: refactor testChannelBalance
2022-11-17 21:06:11 +08:00
yyforyongyu
876bb5c066
itest: refactor testExportChannelBackup
2022-11-17 21:06:11 +08:00
yyforyongyu
40a5f94ef7
lntemp+itest: refactor testChannelBackupUpdates
2022-11-17 21:06:11 +08:00
yyforyongyu
4e821a6e3d
lntemp+itest: refactor testInvoiceSubscriptions
2022-11-17 21:06:11 +08:00
yyforyongyu
d260ac1ab5
itest: refactor testPaymentFollowingChannelOpen
2022-11-17 21:06:11 +08:00
yyforyongyu
e963506f7d
lntemp+itest: refactor testListPayments
2022-11-17 21:06:10 +08:00
