-
Notifications
You must be signed in to change notification settings - Fork 342
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
Handle fallible per commitment point in channel establishment #3109
Open
alecchendev
wants to merge
22
commits into
lightningdevkit:main
Choose a base branch
from
alecchendev:2024-06-async-commit-point-funding
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Handle fallible per commitment point in channel establishment #3109
alecchendev
wants to merge
22
commits into
lightningdevkit:main
from
alecchendev:2024-06-async-commit-point-funding
+496
−228
Commits on Jun 18, 2024
-
Async signing test util follow ups
- Remove unused unavailable signers in TestKeysInterface - Remove unnecessary display implementation for SignerOp - Move set of test disabled signer ops to EnforcementState
Configuration menu - View commit details
-
Copy full SHA for f49df83 - Browse repository at this point
Copy the full SHA f49df83View commit details -
Move clearing signer funding signed flag to signer_unblocked
This makes things more consistent with the overall style we use for async signing. We try to generate a signature for a message, if it's not there, we set a pending flag, and only in `signer_unblocked` we clear the flag and retry the signature/message.
Configuration menu - View commit details
-
Copy full SHA for cbb4a9e - Browse repository at this point
Copy the full SHA cbb4a9eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3f840f2 - Browse repository at this point
Copy the full SHA 3f840f2View commit details -
Configuration menu - View commit details
-
Copy full SHA for df10ff4 - Browse repository at this point
Copy the full SHA df10ff4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3e1cae5 - Browse repository at this point
Copy the full SHA 3e1cae5View commit details -
Configuration menu - View commit details
-
Copy full SHA for cb9ba76 - Browse repository at this point
Copy the full SHA cb9ba76View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6013f0d - Browse repository at this point
Copy the full SHA 6013f0dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3d2f215 - Browse repository at this point
Copy the full SHA 3d2f215View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0c130d9 - Browse repository at this point
Copy the full SHA 0c130d9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4494d96 - Browse repository at this point
Copy the full SHA 4494d96View commit details -
Change get_per_commitment_point to return result type
Includes simple changes to test util signers and tests, as well as handling the error case for get_per_commitment_point in HolderCommitmentPoint. This leaves an `.expect("TODO")` in places that will be addressed in upcoming commits.
Configuration menu - View commit details
-
Copy full SHA for 3c82bb5 - Browse repository at this point
Copy the full SHA 3c82bb5View commit details -
Configuration menu - View commit details
-
Copy full SHA for a4eb312 - Browse repository at this point
Copy the full SHA a4eb312View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1a76f82 - Browse repository at this point
Copy the full SHA 1a76f82View commit details -
Configuration menu - View commit details
-
Copy full SHA for 41e4139 - Browse repository at this point
Copy the full SHA 41e4139View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1e9a36f - Browse repository at this point
Copy the full SHA 1e9a36fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 84f4ff3 - Browse repository at this point
Copy the full SHA 84f4ff3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 02b9f00 - Browse repository at this point
Copy the full SHA 02b9f00View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7bf5c62 - Browse repository at this point
Copy the full SHA 7bf5c62View commit details -
Configuration menu - View commit details
-
Copy full SHA for ccbe92d - Browse repository at this point
Copy the full SHA ccbe92dView commit details -
Configuration menu - View commit details
-
Copy full SHA for db9f2e6 - Browse repository at this point
Copy the full SHA db9f2e6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 68f1d8a - Browse repository at this point
Copy the full SHA 68f1d8aView commit details -
Configuration menu - View commit details
-
Copy full SHA for b798ffa - Browse repository at this point
Copy the full SHA b798ffaView commit details
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.