bitcoindevkit / bitcoindevkit/bdk-message-signer

Support the official BIP test vectors

Open
#5 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
5
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Hello, Thanks for the work.
I skimmed through the repository and I can see some of the code with lack of tests (cmiiw).
I think it'll be worth adding more support for the official BIP 322 test vectors, this could strengthen the implementation.
I know it's still experimental maybe that was already on your roadmap, but yeah I'm using this issue to put an URL in it so that we can follow up.

1 - https://github.com/bitcoin/bips/blob/master/bip-0322/generated-test-vectors.json
2 - https://github.com/bitcoin/bips/blob/master/bip-0322/basic-test-vectors.json

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the linked BIP-322 generated and basic test-vector files, then inspect the repository's current message-signing implementation and tests. Determine which vector cases are already covered and define completion as passing coverage for the applicable official vectors.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cryptography, testing-qa
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.