Skip to content

[Draft] Add new cryptlib and support index4 parameters#21

Draft
optout21 wants to merge 8 commits into
mainfrom
index4_0
Draft

[Draft] Add new cryptlib and support index4 parameters#21
optout21 wants to merge 8 commits into
mainfrom
index4_0

Conversation

@optout21

@optout21 optout21 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Fixes #16 and #2 .

Use cryptlib from external common repository (see #17 ).

Changes for index4:

  • Extend InputReqs with signerIndex4 field (next to signerIndex)
  • Extend Cets with signerIndex4 field (next to signerIndex)
  • Read them from the Json version, if missing, fallback to default 0
  • Extend signHashEcdsa() with extra int signerIndex4 parameter
  • Extend createCetAdaptorSigs() with extra int signingKeyIndex4 parameter
  • Extend signTransaction()with extra int keyIndex4 parameter
  • Pass the extra parameters, down to the crytlib library methods.

Note: The index4 change is untested!

See also:

optout21 added 8 commits June 9, 2026 13:50
Changes done:
- Extend `InputReqs` with `signerIndex4` field (next to `signerIndex`)
- Extend `Cets` with `signerIndex4` field (next to `signerIndex`)
- Read them from the Json version, if missing, fallback to default 0
- Extend `signHashEcdsa()` with extra `int signerIndex4` parameter
- Extend `createCetAdaptorSigs()` with extra `int signingKeyIndex4` parameter
- Extend `signTransaction()`with extra `int keyIndex4` parameter
- Pass the extra parameters, down to the crytlib library methods.
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.

APP - Use cryptlib from common repository

1 participant