Remove KZG verification from local block production and blobs fetched from the EL#7713
Conversation
|
Some required checks have failed. Could you please take a look @jimmygchen? 🙏 |
0d7ea70 to
d7ab88c
Compare
pawanjay176
left a comment
There was a problem hiding this comment.
Just noting here that we don't really gain anything from kzg verifying blobs returned by the builder when publishing because we have already signed the message by the time we receive the BlobsBundle.
|
This pull request has been removed from the queue for the following reason: The merge conditions cannot be satisfied due to failing checks: You can check the last failing draft PR here: #7772. You may have to fix your CI before adding the pull request to the queue again. |
Continuation of #7713, addresses comment about skipping KZG verification on EL fetched blobs: #7713 (comment)
Issue Addressed
#7700
Proposed Changes
As described in title, the EL already performs KZG verification on all blobs when they entered the mempool, so it's redundant to perform extra validation on blobs returned from the EL.
This PR removes
This PR does not remove KZG verification on the block publishing path yet.