-
-
Notifications
You must be signed in to change notification settings - Fork 290
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
feat: implement blob sidecars with inclusion proof instead of signtaures #6089
Conversation
fcc2aec
to
a417362
Compare
Performance Report✔️ no performance regression detected Full benchmark results
|
|
6a0d02c
to
8174dbf
Compare
yes it did, i changed base to just reflect this diff |
860784d
to
17154a8
Compare
1c1f667
to
e8d10c1
Compare
0e793db
to
f43984e
Compare
d5a5a47
to
5799ef2
Compare
packages/beacon-node/src/chain/seenCache/seenGossipBlockInput.ts
Outdated
Show resolved
Hide resolved
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6089 +/- ##
============================================
+ Coverage 80.83% 80.95% +0.11%
============================================
Files 185 185
Lines 17986 17935 -51
Branches 1082 1078 -4
============================================
- Hits 14539 14519 -20
+ Misses 3421 3389 -32
- Partials 26 27 +1 |
40c7561
to
310b871
Compare
modify the api fix validator handling fix val build refactor the beacon node impl for the new blobs architecture get the build working with no blobs fix the blob sidecars transmisssion relocate compute blob sidecars relocate verify merkle branch verify inclusion proof fix tests and types fix unit test update the spec test versions skip newly required merkle proof runner change the minimal/mainnet preset based constant strategy add other constants to tests apply feedback apply feedback relocate utils
1d0e4a7
to
3ceb6bf
Compare
@@ -207,3 +215,13 @@ function validateBlobsAndProofs( | |||
throw Error("Invalid verifyBlobKzgProofBatch"); | |||
} | |||
} | |||
|
|||
function validateInclusionProof(config: ChainForkConfig, blobSidecar: deneb.BlobSidecar): boolean { |
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.
unused config
@@ -37,6 +38,7 @@ export type BlobSidecarErrorType = | |||
| {code: BlobSidecarErrorCode.PARENT_UNKNOWN; parentRoot: RootHex} | |||
| {code: BlobSidecarErrorCode.NOT_LATER_THAN_PARENT; parentSlot: Slot; slot: Slot} | |||
| {code: BlobSidecarErrorCode.PROPOSAL_SIGNATURE_INVALID} | |||
| {code: BlobSidecarErrorCode.INCLUSION_PROOF_INVALID} |
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.
need to include slot, blob index here
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.
looks good to me overall, there are 2 minor points, approving in case @g11tech wants to merge and address them in another PR
thanks @tuyennhv , yes let me address then in a followup |
🎉 This PR is included in v1.14.0 🎉 |
…res (ChainSafe#6089) modify the api fix validator handling fix val build refactor the beacon node impl for the new blobs architecture get the build working with no blobs fix the blob sidecars transmisssion relocate compute blob sidecars relocate verify merkle branch verify inclusion proof fix tests and types fix unit test update the spec test versions skip newly required merkle proof runner change the minimal/mainnet preset based constant strategy add other constants to tests apply feedback apply feedback relocate utils
Implement