feat: validate data column sidecars #7073

matthewkeil · 2024-09-09T07:26:10Z

Motivation

Implements validateDataColumnsSidecars in place of stub function

packages/beacon-node/src/chain/validation/dataColumnSidecar.ts

…nsSidecars

matthewkeil added 3 commits August 26, 2024 00:21

feat: update c-kzg to final DAS version

cc263ee

refactor: use trusted-setup from c-kzg package

2844eeb

feat: implement validateDataColumnsSidecars

f239059

matthewkeil changed the base branch from peerDAS to mkeil/update-ckzg September 9, 2024 07:26

matthewkeil requested a review from g11tech September 9, 2024 07:26

g11tech reviewed Sep 9, 2024

View reviewed changes

packages/beacon-node/src/chain/validation/dataColumnSidecar.ts Outdated Show resolved Hide resolved

g11tech reviewed Sep 9, 2024

View reviewed changes

packages/beacon-node/src/chain/validation/dataColumnSidecar.ts Show resolved Hide resolved

matthewkeil added 2 commits September 9, 2024 17:28

feat: check block and column commitments match

76d8447

test: add unit test for validateDataColumnsSidecars

49b13f5

matthewkeil marked this pull request as ready for review September 10, 2024 04:04

matthewkeil requested a review from a team as a code owner September 10, 2024 04:04

matthewkeil requested a review from g11tech September 10, 2024 04:04

matthewkeil added 2 commits September 10, 2024 03:37

fix: invalid build and update validity condition of validateDataColum…

6f1f025

…nsSidecars

fix: make error messages better

38baa72

Base automatically changed from mkeil/update-ckzg to peerDAS September 16, 2024 10:31

matthewkeil added 2 commits September 17, 2024 10:32

Merge branch 'peerDAS' into mkeil/validate-data-column-sidecars

4f3de52

fix: electra vs peerdas type conflict

c2a3c0b

g11tech approved these changes Sep 17, 2024

View reviewed changes

g11tech merged commit 20ef4c6 into peerDAS Sep 17, 2024
13 of 17 checks passed

g11tech deleted the mkeil/validate-data-column-sidecars branch September 17, 2024 09:33

Provide feedback