Skip to content
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

Zk rpc changes #9

Closed
wants to merge 276 commits into from
Closed

Zk rpc changes #9

wants to merge 276 commits into from

Conversation

hexoscott
Copy link
Collaborator

No description provided.

revitteth and others added 25 commits November 14, 2023 22:04
Co-authored-by: anton@gateway.fm <anton@MacBook-Pro-3.fritz.box>
* feat(zk): tip sync

* feat(zk): tip sync improvements

* datastream bookmark fix (#53)

* datastream bookmark fix

* datastream bookmark fix

* stage_batches error handle

* feat(zk): verifications progress

* added progress and bug fix

* removed tx spam

* feat(zk): tip sync improvements

* feat(zk): tip sync improvements

* batches fix and exec progress percent

* feat(zk): tip sync improvements

* feat(zk): tip sync - initial to verified boundary

* feat(zk): tip sync - initial to verified boundary

* code betterment

* Proxy send transaction (#54)

* wip - proxy send transaction

* added rpc.url flag and sendtx upstream

---------

Co-authored-by: Valentin Staykov <v.staykov@razorlabs.com>

* Update cmd/rpcdaemon/commands/send_transaction.go

Co-authored-by: Igor Mandrigin <mandrigin@users.noreply.github.com>

* Update cmd/rpcdaemon/commands/send_transaction.go

Co-authored-by: Igor Mandrigin <mandrigin@users.noreply.github.com>

* initial infra

* some pr comments

* minor log level changes

* batchNumber and batchNumberByBlockNumber

* merge

* fix after merge renames

* verifiedbatchnumber

* IsBlockConsolidated

* ConsolidatedBlockNumber

* readme

* some rearrangement

* some work on get block

* quick fix to get the code building for zkevm rpc

* adding in L1 sequences stage ready to feed to RPC calls

* progress on zk rpc calls

* handing off zkevm_getBatchByNumber to upstream RPC

* feat(rpc): zkevm namespace enablement and docs

* further work on rpc endpoints data

* store correct tx root hash in the header of a block

* test added for receipts root calculation

* store gas used as part of execution

* further progress on filling in block details for RPC

* some minor progress on RPC

* updates to populate more RPC data

---------

Co-authored-by: Valentin Staykov <79150443+V-Staykov@users.noreply.github.com>
Co-authored-by: Valentin Staykov <v.staykov@razorlabs.com>
Co-authored-by: Igor Mandrigin <mandrigin@users.noreply.github.com>
Co-authored-by: Scott Fairclough <scott@hexosoft.co.uk>
* fix the bloom in receipts for zk block requests

* no null logs in zkevm block calls

* zk genesis fixes

* hack for bloom bug

bloom is ommitted in zknode hash calculations so we need to replicate the same.  This will be fixed in a future hardfork

* test to prove off bloom issue in receipt roots
fixes a nil pointer problem when the tx used in the eridb instance has been committed and the tx is newed up again
* send all dynamic gas fix

* don't take cold access gas if no transfer

* staticcallfix in forkid5

* removed print

* removed debug prints
* datastream client doesn't close on each iteration

* removed debug log
* feat(stream): add stateroot to l2tx

* feat(rpcroots): remove roots files

* feat(rpcroots): consume stateroot from the l2 datastream

* fix(state_domain): temp fix effective gas %

* changed block stateroot to tx stateroot

* feat(rpcroots): remove lfs and update readme

---------

Co-authored-by: Valentin Staykov <v.staykov@razorlabs.com>
Copy link

cla-bot bot commented Dec 13, 2023

We require contributors/corporates @hexoscott to read our Contributor License Agreement, please check the Individual CLA document/Corporate CLA document

Copy link

cla-bot bot commented Dec 14, 2023

We require contributors/corporates @hexoscott to read our Contributor License Agreement, please check the Individual CLA document/Corporate CLA document

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants