-
Notifications
You must be signed in to change notification settings - Fork 335
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
chore(security): Bump gnark version #991
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when testing, Go projects don't build. It seems there has been some API changes
make aggregator_start
Starting Aggregator...
# github.com/crate-crypto/go-kzg-4844/internal/kzg
../go/pkg/mod/github.com/crate-crypto/go-kzg-4844@v1.0.0/internal/kzg/kzg_verify.go:73:20: claimedValueG1Jac.ScalarMultiplicationAffine undefined (type bls12381.G1Jac has no field or method ScalarMultiplicationAffine)%
8a154f7
to
7930e82
Compare
|
…hore/bump_gnark_version
@entropidelic re-importing the gnark packages in the batcher resolved the verification error on my end. |
…hore/bump_gnark_version
after running |
We should specify the new supported gnark version on the docs |
@MauroToscano Docs updated |
Bumps the gnark version from v.0.11.0 -> v.0.14.0 to address security concerns outlined by the gnark team here. Closes #967