ref-fvm
ref-fvm copied to clipboard
feat: FIP-0079: syscall for aggregated bls verification
This PR tries to implement https://github.com/filecoin-project/FIPs/blob/master/FIPS/fip-0079.md without breaking FVM backward compatibility.
(NOTE: This PR is largely based on https://github.com/filecoin-project/ref-fvm/pull/1914)
Closes https://github.com/filecoin-project/ref-fvm/issues/2001
Changes:
- Add
verify_bls_aggregatesyscall - Add cargo feature
verify-signaturethat keepsverify_signaturesyscall for backward compatibility - Add integration test for the
verify-signaturefeature - Update CI to build and check the
verify-signaturefeature
@DrPeterVanNostrand could you please help taking a look at this PR?
Codecov Report
Attention: Patch coverage is 85.71429% with 23 lines in your changes are missing coverage. Please review.
Project coverage is 57.04%. Comparing base (
c39d880) to head (169d615).
Additional details and impacted files
@@ Coverage Diff @@
## master #2003 +/- ##
===========================================
- Coverage 75.62% 57.04% -18.58%
===========================================
Files 157 119 -38
Lines 15750 10381 -5369
===========================================
- Hits 11911 5922 -5989
- Misses 3839 4459 +620
| Files | Coverage Δ | |
|---|---|---|
| fvm/src/gas/price_list.rs | 74.65% <100.00%> (-3.66%) |
:arrow_down: |
| fvm/src/syscalls/crypto.rs | 100.00% <100.00%> (ø) |
|
| fvm/src/syscalls/mod.rs | 97.14% <100.00%> (+0.06%) |
:arrow_up: |
| shared/src/crypto/signature.rs | 53.84% <83.33%> (-9.76%) |
:arrow_down: |
| fvm/src/syscalls/context.rs | 93.97% <72.22%> (-6.03%) |
:arrow_down: |
| fvm/src/kernel/default.rs | 82.13% <76.66%> (-0.20%) |
:arrow_down: |