-
Notifications
You must be signed in to change notification settings - Fork 119
[4/?] StaticAddr: Instant deposit withdrawals #719
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
hieblmi
merged 8 commits into
lightninglabs:static-addr-staging
from
hieblmi:static-addr-4
Jun 5, 2024
Merged
[4/?] StaticAddr: Instant deposit withdrawals #719
hieblmi
merged 8 commits into
lightninglabs:static-addr-staging
from
hieblmi:static-addr-4
Jun 5, 2024
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
309bcdc
to
4611204
Compare
bhandras
reviewed
Apr 5, 2024
c5e9e9d
to
c7d2160
Compare
c204e8d
to
34f843c
Compare
bc540a6
to
1b4ca0c
Compare
f614090
to
70a74bd
Compare
bc8e20c
to
fb1300f
Compare
bb36baf
to
28239df
Compare
fc7f4d1
to
dd75d00
Compare
sputn1ck
approved these changes
Jun 4, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
expiry_sweep_txid BLOB | ||
expiry_sweep_txid BLOB, | ||
|
||
-- withdrawal_sweep_pk_script is the address that will be used to sweep the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: commment is off
28239df
to
dbc351a
Compare
6d65aaf
to
64dd989
Compare
dbc351a
to
c4012d0
Compare
56169ab
to
848cf09
Compare
Deposit schema extension with a withdrawal sweep address. If the user selects a deposit for withdrawal the destination address of the sweep is stored here.
A new rpc for deposit withdrawals is added. A withdrawal cooperatively spends the 2/2 musig deposit outpoint to a client-specified address.
A server endpoint to obtain a partial sig to cooperatively spend a 2/2 musig deposit outpoint.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR enables instant withdrawals of funds at static address deposits, allowing the client to omit the CSV timeout period of their Musig2 outpoints and cooperatively spend selected deposit outpoints cooperatively with the server.
Withdrawing can be done for selected or all eligible deposits. Examples:
Selected deposit:
All deposits: