Lodestar prover for execution api - #5222
Conversation
dapplion
left a comment
There was a problem hiding this comment.
Overall direction looks good! 🔥 looking forward to the polished PR
10f32da to
2f32ebb
Compare
Performance Report✔️ no performance regression detected Full benchmark results
|
|
This PR functionality is ready and tested manually with the example given in the readme file. Now adding the unit and integration tests. Please share your earlier feedback. |
b75105f to
be4557a
Compare
| const verifiedHandler = supportedELRequests[payload.method]; | ||
|
|
||
| if (verifiedHandler !== undefined) { | ||
| return verifiedHandler({payload, handler, rootProvider: proofProvider}); |
Check failure
Code scanning / CodeQL
Unvalidated dynamic method call
|
looks exciting on the first pass 👏 , will need to give more brain/review cycles to the PR |
32498d6 to
07fd574
Compare
g11tech
left a comment
There was a problem hiding this comment.
approving with the caveat that followup PRs would need to polish and address some of the unaddressed issues here to productionize the prover
But an amazing effort here to bring such an amazing functionality to lodestar codebase! 👏
All the remaining points are added as action item to this issue #4706 so we will not forget it. |
|
🎉 This PR is included in v1.8.0 🎉 |
Motivation
Make it easier for third party developers to utilize the light client and verify the non-trusted eth responses.
Description
Create a lightclient prover utility for web3 providers.
Closes #4706
Steps to test or reproduce
Run all tests.