Skip to content
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

static addr loopin: add support for sweepless sweep batching #844

Open
wants to merge 9 commits into
base: static-addr-staging
Choose a base branch
from

Conversation

sputn1ck
Copy link
Member

@sputn1ck sputn1ck commented Nov 4, 2024

This PR adds support for serverside sweepless sweep batching, by listening for notifications from the server and responding
with the signature.

Only the 8 last commits are for review

@sputn1ck sputn1ck marked this pull request as draft November 4, 2024 20:23
@hieblmi hieblmi force-pushed the static-addr-staging branch 3 times, most recently from 60ff75b to ec3070f Compare November 5, 2024 18:22
This commit changes the loopin sweepless sweep
handling, by changing the hardcoded fee messages to
a map of fee versions.
@sputn1ck sputn1ck force-pushed the static_sweep_batching branch 3 times, most recently from 7074ba0 to bbff3d0 Compare November 7, 2024 21:28
This commit adds a new notification to the proto, which is used by the
server to request signatures for a deposit of a static loop in.
This commit adds support for the new notification of type
NotificationTypeStaticLoopInSweepRequest. This notification is sent when
a sweep request is received from the server.
This commit adds the listening for sweep requests from the server.
On a sweep request the loopin manager will fetch the loop in from the
db, do sanity and safety checks and then sign the psbt for the input
and send it back to the server.
This commit updates the loop in FSM to handle the new server based
pulled sweepless sweep sigs.
@sputn1ck sputn1ck marked this pull request as ready for review November 12, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant