Skip to content

Commit

Permalink
chore: remove 1.18 support (#70)
Browse files Browse the repository at this point in the history
  • Loading branch information
kevaundray authored Jun 20, 2024
1 parent 886d117 commit e13c8be
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 deletions.
1 change: 0 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
14 changes: 6 additions & 8 deletions readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -21,30 +21,28 @@ this library.
To run the benchmarks, execute the following command:

```
$ go test -bench=.
go test -bench=.
```

## Consensus specs

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 [email protected].
- For security bugs in this library, email <[email protected]>.
- 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
Expand All @@ -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

Expand Down

0 comments on commit e13c8be

Please sign in to comment.