decentralized-identity / bbs-signature

The BBS Signature Scheme
https://identity.foundation/bbs-signature/draft-irtf-cfrg-bbs-signatures.html
Apache License 2.0
80 stars 26 forks source link

Update test vectors #273

Closed BasileiosKal closed 1 year ago

BasileiosKal commented 1 year ago

Test vectors for #267 and #268.

Breaking changes from the PRs

  1. From #267

    • hash_to_scalar: removed count. There is no longer a loop and the input to expand_message is just msg_octets instead of msg_prime = msg_octets || I2OSP(count, 8)
    • creat_generators: removed n (and the check at step 8) and went from I2OSP(n, 4) to I2OSP(i, 8) in the input to expand_message when calculating v.
  2. From #268

    • ciphersuite_id: added HM2S_ at the end (the MAP_TO_SCALAR_ID of the map_to_scalar operation).
BasileiosKal commented 1 year ago

Note that the test vectors use variable length messages now (with one of the messages being empty). Addresses one of the comments in #223.

BasileiosKal commented 1 year ago

Discussed on the WG call of the 10th of July. Fixtures are cross validated. Mergin