Skip to content

Tests checking that TrezorSigner produces the same signatures as SoftwareSigner #1928

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

Open
wants to merge 1 commit into
base: fill_order_dest_tests
Choose a base branch
from

Conversation

ImplOfAnImpl
Copy link
Contributor

Instead of accepting impl Rng + CryptoRng directly, signing functions now accept a more specific SigAuxDataProvider, which returns auxiliary data for bip-340 nonce generation:

  1. using random auxiliary data is what we did before and still do in the production code;
  2. using all zeros for the data is what we do in the Trezor firmware.

So, SoftwareSigner can now be parameterized with a SigAuxDataProvider, which allows the tests to force it to produce trezor-like signatures, so that TrezorSigner's signatures can be compared against them.

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.

1 participant