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

StaticAddr: merge staging to master #853

Open
wants to merge 68 commits into
base: master
Choose a base branch
from
Open

Conversation

hieblmi
Copy link
Collaborator

@hieblmi hieblmi commented Nov 19, 2024

This PR merges the static address loop-in feature to the main repository branch.

@hieblmi hieblmi force-pushed the static-addr-staging branch 3 times, most recently from 5f09927 to fa759a9 Compare November 19, 2024 12:29
@hieblmi hieblmi self-assigned this Nov 19, 2024
hieblmi and others added 26 commits November 21, 2024 16:10
We add CRU procedures for static address swaps.
The static address state machine adds states
to reflect the state of a deposit during a
loop-in swap.
In this commit we introduce maximum fee percentages
for the static loop-in htlc transactions. Since
the server has the ability to publish htlc transactions
without settling the swap payment we have to restrict
the amount the server allocates for fees of these
transactions.
In this commit we add the static address
loop-in state machine and its orchestration
through the manager.
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
@lightninglabs-deploy
Copy link

@bhandras: review reminder
@starius: review reminder
@sputn1ck: review reminder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants