Elias Rohrer [Mon, 28 Aug 2023 11:57:29 +0000 (13:57 +0200)]
Set `payment_secret` when sending probes
Previously, we'd leave the payment secret field empty while sending
probes, which resulted in having them rejected
with `(PERM|invalid_onion_payload)` by Eclair nodes.
In order to mitigate the issue, we just set a random payment secret.
Elias Rohrer [Mon, 21 Aug 2023 09:56:34 +0000 (11:56 +0200)]
Remove mention of spontaneous payments from `lightning-invoice`
Elias Rohrer [Tue, 12 Sep 2023 11:37:57 +0000 (13:37 +0200)]
Cleanup `ChannelId` re-export
`ChannelId` was weirdly listed in the re-export section of the docs and
reachable via multiple paths. Here we opt to make the `channel_id`
module private and leave only the `ChannelId` struct itself exposed.
Valentine Wallace [Sun, 10 Sep 2023 04:42:18 +0000 (00:42 -0400)]
Only allow creating 1-hop blinded paths.
Useful until forwarding and receiving to multi-hop blinded paths is supported.
Valentine Wallace [Sat, 9 Sep 2023 23:36:18 +0000 (19:36 -0400)]
Test sending and receiving to 1-hop blinded paths
Valentine Wallace [Sun, 10 Sep 2023 05:22:49 +0000 (01:22 -0400)]
Support receiving to 1-hop blinded payment paths.
Valentine Wallace [Sun, 10 Sep 2023 05:10:03 +0000 (01:10 -0400)]
Parameterize InboundPayload reads with NodeSigner
This will be used in the next commit to deserialize encrypted TLVs for
receiving to 1-hop blinded paths.
Valentine Wallace [Thu, 13 Jul 2023 02:32:06 +0000 (22:32 -0400)]
Support paying blinded paths.
Matt Corallo [Mon, 28 Aug 2023 01:25:36 +0000 (01:25 +0000)]
Restrict `ChannelManager` persist in fuzzing to when we're told to
In the `chanmon_consistency` fuzz, we currently "persist" the
`ChannelManager` on each loop iteration. With the new logic in the
past few commits to reduce the frequency of `ChannelManager`
persistences, this behavior now leaves a gap in our test coverage -
missing persistence notifications.
In order to cath (common-case) persistence misses, we update the
`chanmon_consistency` fuzzer to no longer persist the
`ChannelManager` unless the waker was woken and signaled to
persist, possibly reloading with a previous `ChannelManager` if we
were not signaled.
Matt Corallo [Tue, 12 Sep 2023 21:24:41 +0000 (21:24 +0000)]
Merge pull request #2169 from TheBlueMatt/2023-03-monitor-e-monitor
Block the mon update removing a preimage until upstream mon writes
David Caseria [Tue, 12 Sep 2023 20:30:58 +0000 (16:30 -0400)]
Make HTLCDescriptor writeable
Matt Corallo [Mon, 28 Aug 2023 01:35:16 +0000 (01:35 +0000)]
Remove largely useless checks in chanmon_consistency fuzzer
When reloading nodes A or C, the chanmon_consistency fuzzer
currently calls `get_and_clear_pending_msg_events` on the node,
potentially causing additional `ChannelMonitor` or `ChannelManager`
updates, just to check that no unexpected messages are generated.
There's not much reason to do so, the fuzzer could always swap for
a different command to call the same method, and the additional
checking requires some weird monitor persistence introspection.
Here we simplify the fuzzer by simply removing this logic.
Matt Corallo [Thu, 24 Aug 2023 20:02:08 +0000 (20:02 +0000)]
Skip persistence in the usual case handling channel_reestablish
When we handle an inbound `channel_reestablish` from our peers it
generally doesn't change any state and thus doesn't need a
`ChannelManager` persistence. Here we avoid said persistence where
possible.
Matt Corallo [Thu, 24 Aug 2023 19:57:45 +0000 (19:57 +0000)]
Always persist the `ChannelManager` on a failed ChannelUpdate
If we receive a `ChannelUpdate` message which was invalid, it can
cause us to force-close the channel, which should result in a
`ChannelManager` persistence, though its not critical to do so.
Matt Corallo [Sun, 10 Sep 2023 23:10:03 +0000 (23:10 +0000)]
Avoid persisting `ChannelManager` in response to peer connection
When a peer connects and we send some `channel_reestablish`
messages or create a `per_peer_state` entry there's really no
reason to need to persist the `ChannelManager`. None of the
possible actions we take immediately result in a change to the
persisted contents of a `ChannelManager`, only the peer's later
`channel_reestablish` message does.
Matt Corallo [Thu, 24 Aug 2023 19:36:58 +0000 (19:36 +0000)]
Move a handful of channel messages to notify-without-persist
Many channel related messages don't actually change the channel
state in a way that changes the persisted channel. For example,
an `update_add_htlc` or `update_fail_htlc` message simply adds the
change to a queue, changing the channel state when we receive a
`commitment_signed` message.
In these cases there's really no reason to wake the background
processor at all - there's no response message and there's no state
update. However, note that if we close the channel we should
persist the `ChannelManager`. If we send an error message without
closing the channel, we should wake the background processor
without persisting.
Here we move to the appropriate `NotifyOption` on some of the
simpler channel message handlers.
Matt Corallo [Thu, 24 Aug 2023 18:37:18 +0000 (18:37 +0000)]
Update `channelmanager::NotifyOption` to indicate persist or event
As we now signal events-available from persistence-needed
separately, the `NotifyOption` enum should include a separate
variant for events-but-no-persistence, which we add here.
Matt Corallo [Fri, 8 Sep 2023 20:26:29 +0000 (20:26 +0000)]
Separate ChannelManager needing persistence from having events
Currently, when a ChannelManager generates a notification for the
background processor, any pending events are handled and the
ChannelManager is always re-persisted.
Many channel related messages don't actually change the channel
state in a way that changes the persisted channel. For example,
an `update_add_htlc` or `update_fail_htlc` message simply adds the
change to a queue, changing the channel state when we receive a
`commitment_signed` message.
In these cases we shouldn't be re-persisting the ChannelManager as
it hasn't changed (persisted) state at all. In anticipation of
doing so in the next few commits, here we make the public API
handle the two concepts (somewhat) separately. The notification
still goes out via a single waker, however whether or not to
persist is now handled via a separate atomic bool.
Matt Corallo [Mon, 11 Sep 2023 03:38:14 +0000 (03:38 +0000)]
Make it harder to forget to call CM::process_background_events
Prior to any actions which may generate a `ChannelMonitorUpdate`,
and in general after startup,
`ChannelManager::process_background_events` must be called. This is
mostly accomplished by doing so on taking the
`total_consistency_lock` via the `PersistenceNotifierGuard`. In
order to skip this call in block connection logic, the
`PersistenceNotifierGuard::optionally_notify` constructor did not
call the `process_background_events` method.
However, this is very easy to misuse - `optionally_notify` does not
convey to the reader that they need to call
`process_background_events` at all.
Here we fix this by adding a separate
`optionally_notify_skipping_background_events` method, making the
requirements much clearer to callers.
Matt Corallo [Wed, 5 Jul 2023 16:15:59 +0000 (16:15 +0000)]
Test monitor update completion actions on pre-startup completion
This adds a test for monitor update actions being completed on
startup if a monitor update completed "while we were shut down"
(or, really, the manager didn't get persisted after the update
completed).
Matt Corallo [Tue, 11 Jul 2023 19:49:41 +0000 (19:49 +0000)]
Split `expect_payment_forwarded` into a function called by macro
Also allowing us to pass the event manually.
Matt Corallo [Mon, 21 Aug 2023 18:44:22 +0000 (18:44 +0000)]
Update tests to test re-claiming of forwarded HTLCs on startup
Because some of these tests require connecting blocks without
calling `get_and_clear_pending_msg_events`, we need to split up
the block connection utilities to only optionally call
sanity-checks.
Matt Corallo [Sun, 27 Aug 2023 20:37:36 +0000 (20:37 +0000)]
Correct `expect_payment_forwarded` upstream channel checking
`expect_payment_forwarded` takes a bool to indicate that the
inbound channel on which we received a forwarded payment has been
closed, but then ignores it in favor of looking at the fee in the
event. While this is generally correct, in cases where we process
an event after a channel was closed, which was generated before a
channel closed this is incorrect.
Instead, we examine the bool we already passed and use that.
Matt Corallo [Thu, 7 Sep 2023 02:22:52 +0000 (02:22 +0000)]
Block the mon update removing a preimage until upstream mon writes
When we forward a payment and receive an `update_fulfill_htlc`
message from the downstream channel, we immediately claim the HTLC
on the upstream channel, before even doing a `commitment_signed`
dance on the downstream channel. This implies that our
`ChannelMonitorUpdate`s "go out" in the right order - first we
ensure we'll get our money by writing the preimage down, then we
write the update that resolves giving money on the downstream node.
This is safe as long as `ChannelMonitorUpdate`s complete in the
order in which they are generated, but of course looking forward we
want to support asynchronous updates, which may complete in any
order.
Thus, here, we enforce the correct ordering by blocking the
downstream `ChannelMonitorUpdate` until the upstream one completes.
Like the `PaymentSent` event handling we do so only for the
`revoke_and_ack` `ChannelMonitorUpdate`, ensuring the
preimage-containing upstream update has a full RTT to complete
before we actually manage to slow anything down.
Matt Corallo [Thu, 31 Aug 2023 19:06:34 +0000 (19:06 +0000)]
Clean up test handling of resending responding commitment_signed
When we need to rebroadcast a `commitment_signed` on reconnect in
response to a previous update (ie not one which contains any
updates) we previously hacked in support for it by passing a `-1`
for the number of expected update_add_htlcs. This is a mess, and
with the introduction of `ReconnectArgs` we can now clean it up
easily with a new bool.
Matt Corallo [Thu, 31 Aug 2023 18:47:13 +0000 (18:47 +0000)]
Fix various unused warnings in test and regular builds
Matt Corallo [Mon, 11 Sep 2023 23:15:49 +0000 (23:15 +0000)]
Merge pull request #2567 from G8XSU/payment-id
Add PaymentId in ChannelManager.list_recent_payments()
Matt Corallo [Mon, 11 Sep 2023 20:20:50 +0000 (20:20 +0000)]
Merge pull request #2566 from G8XSU/check-best-block-2538
Verify channel-monitor processes blocks with skipped best_block
Gursharan Singh [Sun, 10 Sep 2023 06:56:45 +0000 (23:56 -0700)]
Add PaymentId in ChannelManager.list_recent_payments()
valentinewallace [Mon, 11 Sep 2023 17:59:05 +0000 (13:59 -0400)]
Merge pull request #2563 from tnull/2023-09-kvstore-followups
`KVStore` upstreaming followups
Elias Rohrer [Fri, 8 Sep 2023 08:00:25 +0000 (10:00 +0200)]
Add length check for read ChannelMonitor keys
Matt Corallo [Mon, 11 Sep 2023 03:10:36 +0000 (03:10 +0000)]
Make PersistenceNotifierGuard::optionally_notify take a ChanMan ref
Long ago, for reasons lost to the ages, the
`PersistenceNotifierGuard::optionally_notify` constructor didn't
take a `ChannelManager` reference, but rather explicit references
to the fields of it that it needs.
This is cumbersome and useless, so we fix it here.
Matt Corallo [Thu, 24 Aug 2023 18:34:55 +0000 (18:34 +0000)]
Rename `ChannelManager` update future methods for new API
In the next commit, we separate `ChannelManager`'s concept of
waking a listener to both be persisted and to allow the user to
handle events. Here we rename the future-fetching method in
anticipation of this split.
Matt Corallo [Sun, 10 Sep 2023 03:02:22 +0000 (03:02 +0000)]
Merge pull request #2514 from valentinewallace/2023-08-compute-blindedpayinfo
Aggregate `BlindedPayInfo` for blinded routes
Matt Corallo [Sat, 9 Sep 2023 18:54:06 +0000 (18:54 +0000)]
Merge pull request #2495 from dunxen/2023-07-channelenummap
Use a single peer state map for all channel phases in peer state
Gursharan Singh [Fri, 8 Sep 2023 22:15:16 +0000 (15:15 -0700)]
Verify channel-monitor processes blocks with skipped best_block
This can happen due to races b/w client's call to block_connect
and adding newly created channel-monitor to chain-monitor using
watch_channel in funding_created.
Duncan Dean [Mon, 14 Aug 2023 11:28:47 +0000 (13:28 +0200)]
Remove v1 peer state channel maps & refactor with ChannelPhase
Duncan Dean [Mon, 14 Aug 2023 09:28:40 +0000 (11:28 +0200)]
Refactor `ChannelManager` with `ChannelPhase`
Matt Corallo [Fri, 8 Sep 2023 20:08:08 +0000 (20:08 +0000)]
Merge pull request #2549 from yanganto/socket-addr
Rename SocketAddress from NetAddress
Valentine Wallace [Thu, 7 Sep 2023 21:11:30 +0000 (17:11 -0400)]
Struct-ify blinded payment path intermediate node info
Valentine Wallace [Thu, 7 Sep 2023 21:10:21 +0000 (17:10 -0400)]
Remove unnecessary doc links
Valentine Wallace [Thu, 24 Aug 2023 19:23:06 +0000 (15:23 -0400)]
Fix blinded payment TLV ser to not length-prefix
impl_writeable_tlv_based includes a length prefix to the TLV stream, which we
don't want.
Valentine Wallace [Thu, 7 Sep 2023 17:32:51 +0000 (13:32 -0400)]
Derive Clone and Debug for blinded payment TLV structs
Valentine Wallace [Thu, 7 Sep 2023 17:32:30 +0000 (13:32 -0400)]
Make blinded payment TLV fields public.
These should've been made public when they were added for use in
BlindedPath::new_for_payment.
Valentine Wallace [Sat, 19 Aug 2023 21:56:33 +0000 (17:56 -0400)]
Support aggregating htlc_maximum_msat for BlindedPayInfo
Valentine Wallace [Sat, 19 Aug 2023 21:12:15 +0000 (17:12 -0400)]
Support aggregating htlc_minimum_msat for BlindedPayInfo
Valentine Wallace [Wed, 21 Jun 2023 00:27:57 +0000 (20:27 -0400)]
Compute aggregated BlindedPayInfo in path construction
Antonio Yang [Mon, 4 Sep 2023 02:34:21 +0000 (10:34 +0800)]
Rename SocketAddress from NetAddress
Duncan Dean [Wed, 26 Jul 2023 09:20:08 +0000 (11:20 +0200)]
Introduce `ChannelPhase` enum
We introduce the `ChannelPhase` enum which will contain the different
channel structs wrapped by each of its variants so that we can place
these within a single `channel_by_id` map in `peer_state` in the
following commits. This will reduce the number of map lookup operations
we need to do in `ChannelManager`'s various methods. It will also make
certain channel counting logic easier to reason about with less risk of
forgetting to modify logic when new channels structs are introduced for
V2 channel establishment.
Duncan Dean [Thu, 24 Aug 2023 13:32:05 +0000 (15:32 +0200)]
Remove outdated `Channel` TODO
Elias Rohrer [Fri, 8 Sep 2023 07:40:45 +0000 (09:40 +0200)]
Have `path_to_windows_str` take reference to avoid `clone`s
Matt Corallo [Fri, 8 Sep 2023 01:52:38 +0000 (01:52 +0000)]
Merge pull request #2371 from jkczyz/2023-06-offer-message-handling
BOLT 12 Invoice payments
Matt Corallo [Thu, 7 Sep 2023 22:26:03 +0000 (22:26 +0000)]
Merge pull request #2472 from tnull/2023-08-add-kvstore
Replace `KVStorePersister` with `KVStore`
Jeffrey Czyz [Tue, 5 Sep 2023 19:32:53 +0000 (14:32 -0500)]
Configure BOLT 12 invoice payment retry strategy
Replace a constant three retry attempts for BOLT 12 invoice payments
with a retry strategy specified when creating a pending outbound
payment. This is configured by users in a later commit when constructing
an InvoiceRequest or a Refund.
Jeffrey Czyz [Tue, 5 Sep 2023 20:21:35 +0000 (15:21 -0500)]
Use u32 instead of usize in Retry::Attempts
An upcoming commit requires serializing Retry, so use a type with a
fixed byte length. Otherwise, using eight bytes to serialize a usize
would fail to read on 32-bit machines.
Jeffrey Czyz [Thu, 31 Aug 2023 22:22:31 +0000 (17:22 -0500)]
Add tests for send_payment_for_bolt12_invoice
Jeffrey Czyz [Thu, 31 Aug 2023 22:19:29 +0000 (17:19 -0500)]
pub(crate) visibility for offers/test_utils.rs
The test utilities for Offers are needed for testing message handling in
ChannelManager and OutboundPayments.
Jeffrey Czyz [Fri, 1 Sep 2023 19:01:24 +0000 (14:01 -0500)]
Support paying BOLT 12 invoices
Add a send_payment_for_bolt12_invoice method to OutboundPayments for
initiating payment of a BOLT 12 invoice. This will be called from an
OffersMessageHandler, after which any retries are handled using the
Retryable logic.
Jeffrey Czyz [Wed, 6 Sep 2023 19:31:57 +0000 (14:31 -0500)]
Rename OutboundPayments::retry_payment_internal
It will be used for initial attempts at paying BOLT 12 invoices, so
rename it something that covers both that and retries.
Jeffrey Czyz [Fri, 1 Sep 2023 21:23:27 +0000 (16:23 -0500)]
Refactor OutboundPayments::retry_payment_internal
Consolidate the creation and insertion of onion_session_privs to the
PendingOutboundPayment::Retryable arm. In an upcoming commit, this
method will be reused for an initial BOLT 12 invoice payment. However,
onion_session_privs are created using a different helper.
Jeffrey Czyz [Wed, 30 Aug 2023 17:01:15 +0000 (12:01 -0500)]
Add PendingOutboundPayment::InvoiceReceived
When a BOLT 12 invoice has been received, a payment attempt is made and
any errors result in abandoning the PendingOutboundPayment. This results
in generating at PaymentFailed event, which has a PaymentHash. Thus,
when receiving an invoice, transition from AwaitingInvoice to a new
InvoiceReceived state, the latter of which contains a PaymentHash such
the abandon_payment helper can still be used.
Jeffrey Czyz [Wed, 6 Sep 2023 20:17:01 +0000 (15:17 -0500)]
Test removing abandoned AwaitingInvoice payments
Jeffrey Czyz [Thu, 31 Aug 2023 01:22:18 +0000 (20:22 -0500)]
Test for removing stale AwaitingInvoice payments
Jeffrey Czyz [Wed, 6 Sep 2023 18:56:46 +0000 (13:56 -0500)]
Move IDEMPOTENCY_TIMEOUT_TICKS to where it is used
Jeffrey Czyz [Mon, 17 Jul 2023 21:55:22 +0000 (16:55 -0500)]
Add PendingOutboundPayment::AwaitingInvoice
When a BOLT 12 invoice has been requested, in order to guarantee
invoice payment idempotency the future payment needs to be tracked. Add
an AwaitingInvoice variant to PendingOutboundPayment such that only
requested invoices are paid only once. Timeout after a few timer ticks
if a request has not been received.
Elias Rohrer [Mon, 21 Aug 2023 14:17:35 +0000 (16:17 +0200)]
Add `TestStore` implementation of `KVStore`
Elias Rohrer [Fri, 4 Aug 2023 09:34:45 +0000 (11:34 +0200)]
Add benchmarking for `FilesystemStore`
We re-add benchmarking for `FilesystemStore` now that we switched over
to it.
Elias Rohrer [Fri, 4 Aug 2023 14:27:39 +0000 (16:27 +0200)]
Migrate `FilesystemPersister` tests to `FilesystemStore`
Elias Rohrer [Tue, 1 Aug 2023 11:37:46 +0000 (13:37 +0200)]
Migrate to `KVStore`/`FilesystemStore`
Firstly, we switch our BP over to use `FilesystemStore`, which also gives us test
coverage and ensures the compatibility.
Then, we remove the superseded `KVStorePersister` trait and
the `FilesystemPersister` code.
Elias Rohrer [Fri, 4 Aug 2023 14:20:50 +0000 (16:20 +0200)]
Add `read_channel_monitors` utility
This replaces the `FilesystemPersister::read_channelmonitors` method, as
we can now implement a single utility for all `KVStore`s.
Elias Rohrer [Tue, 1 Aug 2023 08:46:51 +0000 (10:46 +0200)]
Add `FilesystemStore`
We upstream the `FilesystemStore` implementation, which is backwards
compatible with `lightning-persister::FilesystemPersister`.
Elias Rohrer [Fri, 4 Aug 2023 08:09:55 +0000 (10:09 +0200)]
Add `test_utils`
We add a utility function needed by upcoming `KVStore` implementation
tests.
Elias Rohrer [Mon, 21 Aug 2023 11:13:56 +0000 (13:13 +0200)]
Update `lightning-persister` crate
Elias Rohrer [Tue, 1 Aug 2023 08:24:53 +0000 (10:24 +0200)]
Add `KVStore` interface trait
We upstream the `KVStore` interface trait from LDK Node, which will
replace `KVStorePersister` in the coming commits.
Besides persistence, `KVStore` implementations will also offer to `list`
keys present in a given `namespace` and `read` the stored values.
Jeffrey Czyz [Tue, 18 Jul 2023 19:41:50 +0000 (14:41 -0500)]
Add an InvoiceRequestFailed event
When an invoice is requested but either receives an error or never
receives a response, surface an event to indicate to the user that the
corresponding future payment has failed.
Elias Rohrer [Thu, 7 Sep 2023 12:16:25 +0000 (14:16 +0200)]
Merge pull request #2134 from jbesraa/add_fromstr_to_netaddress
implement fromstr trait to netaddress
Matt Corallo [Thu, 7 Sep 2023 04:10:59 +0000 (04:10 +0000)]
Merge pull request #2557 from waterson/stale-monitor-force-close-debugging
Log each condition that was violated for a stale monitor
Chris Waterson [Wed, 6 Sep 2023 22:10:51 +0000 (15:10 -0700)]
Fix misspelling, s/and/but/.
Matt Corallo [Wed, 6 Sep 2023 20:22:08 +0000 (20:22 +0000)]
Merge pull request #2555 from tnull/2023-08-have-get-route-take-params
Have `get_route` and `Route` take `RouteParameters`
Elias Rohrer [Wed, 6 Sep 2023 09:26:13 +0000 (11:26 +0200)]
Improve `Route::get_total_amount` docs
Elias Rohrer [Wed, 6 Sep 2023 09:17:51 +0000 (11:17 +0200)]
Include overpaid value in `Route::get_total_fees`
Elias Rohrer [Thu, 31 Aug 2023 13:10:09 +0000 (15:10 +0200)]
Have `Route` hold `RouteParameters`
Elias Rohrer [Thu, 31 Aug 2023 10:25:38 +0000 (12:25 +0200)]
Have `get_route` take `RouteParameters`
jbesraa [Fri, 25 Aug 2023 02:57:56 +0000 (05:57 +0300)]
Implement `from_str` trait for `NetAddress`
- Add fuzz test for `NetAddress` `from_str` function
Chris Waterson [Wed, 6 Sep 2023 15:38:08 +0000 (08:38 -0700)]
Log each condition that was violated for a stale monitor
There are several conditions that can be violated which indicate a stale
monitor. This logs each that doesn't hold.
jbesraa [Mon, 4 Sep 2023 15:29:19 +0000 (18:29 +0300)]
Fix indent in `message_signing.rs`
jbesraa [Fri, 25 Aug 2023 02:57:06 +0000 (05:57 +0300)]
Move `zbase32` implementation to `base32` file
jbesraa [Fri, 25 Aug 2023 02:52:38 +0000 (05:52 +0300)]
Add RFC4648 base32 `encode` and `decode` functions
Matt Corallo [Tue, 5 Sep 2023 16:56:35 +0000 (16:56 +0000)]
Merge pull request #1924 from benthecarman/handle-coinbase-funding-channel
Handle if funding output is in a coinbase transaction
Elias Rohrer [Tue, 5 Sep 2023 07:19:27 +0000 (09:19 +0200)]
Merge pull request #2548 from JosephGoulden/chain-monitor-logging
Downgrade log message "Channel Monitor sync is still in progress" from info to debug
Duncan Dean [Fri, 1 Sep 2023 09:13:57 +0000 (11:13 +0200)]
Add test for coinbase funding transactions
benthecarman [Wed, 19 Apr 2023 08:53:54 +0000 (03:53 -0500)]
Support creating coinbase funding transactions in tests
benthecarman [Mon, 19 Dec 2022 08:02:22 +0000 (02:02 -0600)]
Handle if funding output is in a coinbase transaction
Joseph Goulden [Sun, 3 Sep 2023 11:48:56 +0000 (12:48 +0100)]
Downgrade log message regarding Channel Monitor sync still being in progress from info to debug
Matt Corallo [Fri, 1 Sep 2023 18:25:53 +0000 (18:25 +0000)]
Merge pull request #2522 from sr-gi/202308-set-feature
Adds a set of convenience methods to set non-custom features
Matt Corallo [Fri, 1 Sep 2023 15:13:54 +0000 (15:13 +0000)]
Merge pull request #2543 from optout21/txabort-typo
[minor] Fix wrong msg type in SendTxAbort, typo
Sergi Delgado Segura [Fri, 25 Aug 2023 08:30:07 +0000 (10:30 +0200)]
Adds a set of convenience methods to set non-custom features
Currently only custom features can be set by specifying the feature bit. Add also the
ability to do so for regular features.
Elias Rohrer [Thu, 31 Aug 2023 14:04:33 +0000 (16:04 +0200)]
Add constructor to `RouteParameters`
optout [Thu, 31 Aug 2023 22:00:13 +0000 (00:00 +0200)]
Fix wrong msg type in SendTxAbort, typo
Matt Corallo [Tue, 29 Aug 2023 19:29:21 +0000 (19:29 +0000)]
Merge pull request #2468 from jkczyz/2023-08-offer-payment-id
Offer outbound payments