From e13c8bea2143061aed2608216da21ec011d3f0f8 Mon Sep 17 00:00:00 2001 From: kevaundray Date: Thu, 20 Jun 2024 22:25:20 +0100 Subject: [PATCH] chore: remove 1.18 support (#70) --- .github/workflows/test.yml | 1 - readme.md | 14 ++++++-------- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 05d9e21..6fc8a41 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -18,7 +18,6 @@ jobs: - macos-latest - windows-latest go-version: - - 1.18.x # Minimmum version - 1.21.x # Current version - 1.22.x # Latest version diff --git a/readme.md b/readme.md index 81ed839..fc7372d 100644 --- a/readme.md +++ b/readme.md @@ -5,10 +5,10 @@ is not familiar with EIP-4844, you can think of this library as a KZG library where the polynomial degree is set to 4095 and opening proofs are computed on polynomials in lagrange form (4096 evaluations at 4096'th roots of unity). -## Installation +## Installation ``` -$ go get github.com/crate-crypto/go-kzg-4844 +go get github.com/crate-crypto/go-kzg-4844 ``` ## Example @@ -21,7 +21,7 @@ this library. To run the benchmarks, execute the following command: ``` -$ go test -bench=. +go test -bench=. ``` ## Consensus specs @@ -29,22 +29,20 @@ $ go test -bench=. This version of the code is conformant with the consensus-specs as of the following commit: `017a8495f7671f5fff2075a9bfc9238c1a0982f8` - ## Security -- For security bugs in this library, email kev@the.dev. +- For security bugs in this library, email . - This library uses [gnark-crypto](https://github.com/ConsenSys/gnark-crypto/tree/master) for elliptic curve operations. An audit of gnark can be seen [here](https://github.com/ConsenSys/gnark-crypto/blob/master/audit_oct2022.pdf). - This library uses a more recent version than the audited version, since that + This library uses a more recent version than the audited version, since that version had a serialization bug. We only rely on gnark-crypto's underlying group operations and pairing code for bls12-381. For code that we do need to use, that has not been audited, we have copied it into this library so that it can be a part of this libraries audit. We have noted in the comments which functions we have done this for. - ### Panics Panics can be a DoS vector when running code in a node. This library endeavors @@ -55,7 +53,7 @@ to only panic on startup; only methods which are called when we create the Because we use generics, the minimum golang version needs to be 1.18 or above. Since Golang only back ports security fixes to the latest version and one version behind latest, this library will at most be one version behind latest. -Tests are ran against the 1.18, the current version and the latest version. If support for 1.18 is broken, we will update the CI to reflect this with a given reason. +Tests are ran against the current version and the latest version. ## License