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

Save signatures with more information to db #508

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Conversation

atacann
Copy link

@atacann atacann commented Feb 6, 2025

Description

-While creating TxHandler add information about spend path of input (keypath or script path idx), tag every input with an enum so that we can easily understand which saved nofn signature corresponds to which utxo.
-Save signatures with their tag to db, each row in signatures db is denoted by deposit_outpoint (converted to a unique int), operator_idx, sequential_collateral_idx and kickoff_idx.

Linked Issues

atacann and others added 3 commits February 6, 2025 11:58
…ath is used during txhandler creation, and to store txin type while storing signature to db

Co-authored-by: Mehmet Efe Akça <mmtftr@users.noreply.github.com>
Co-authored-by: Mehmet Efe Akça <mmtftr@users.noreply.github.com>
Co-authored-by: Mehmet Efe Akça <mmtftr@users.noreply.github.com>
@mmtftr mmtftr self-assigned this Feb 6, 2025
@mmtftr mmtftr marked this pull request as ready for review February 6, 2025 14:27
Copy link
Member

@ceyhunsen ceyhunsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But can't we rename NormalSignature to OperatorSignature or NofNSignature, if I am not missing something? NormalSignature doesn't ring a bell for me TBH.

@atacann
Copy link
Author

atacann commented Feb 6, 2025

LGTM. But can't we rename NormalSignature to OperatorSignature or NofNSignature, if I am not missing something? NormalSignature doesn't ring a bell for me TBH.

Yes NormalSignature is not descriptive enough. In this case NormalSignature can include both operator and NofN sigs. It means that there is only one signature per one kickoff, but the other type (WatchtowerSignature) has sigs for each watchtower so they need an additional watchtower_idx parameter.

@mmtftr
Copy link

mmtftr commented Feb 7, 2025

Will rename to SingularSignature in a future PR.

@mmtftr mmtftr merged commit 4e3372e into dev Feb 7, 2025
9 checks passed
@mmtftr mmtftr deleted the atacan/signature_enum branch February 7, 2025 11:55
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.

3 participants