Replies: 1 comment
-
Then you can safely make Also, consider contributing bls377 to the repo! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I have been using @noble/curves v1.2.0 to extend the BLS class to BLS12-377. It works quite well. But today, when I wanted to update the package to v1.4.0, there are type errors because ShortSignatures were added and are required in BLS. I don't need signatures in my application, it seems weird that they are compulsory in the class.
Also, the missing functions are just serialisation of signatures (to/from hex/bytes). Why does the short signature implementation need to re-implement the same functionalities in the EC groups? Aren't the signatures group elements themselves?
E.g.
noble-curves/src/bls12-381.ts
Lines 1201 to 1216 in e14e37a
and
noble-curves/src/bls12-381.ts
Lines 1152 to 1181 in e14e37a
are very similar. Will there be an issue if
ShortSignature.fromHex
just useG1.fromBytes
(subject to refactoring)?Beta Was this translation helpful? Give feedback.
All reactions