proxy: txs, rxs and logs #3457

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.

Sign up for GitHub

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

Jump to bottom

Open

chirag-parmar wants to merge 15 commits into master from vp-transactions

Contributor

chirag-parmar commented Jul 7, 2025 •

edited

Loading

This PR adds support for block based RPC endpoints by adding the following:

helper functions to get and verify receipts rpc/receipts.nim
adds transaction, receipts and logs based JSON RPC method handlers (check this)
adds test cases that do a basic sanity check for the API methods implemented

chirag-parmar changed the title ~~txs, receipts and logs~~ proxy: txs, receipts and logs

chirag-parmar changed the title ~~proxy: txs, receipts and logs~~ proxy: txs, rxs and logs

chirag-parmar marked this pull request as ready for review

July 7, 2025 14:12

chirag-parmar force-pushed the vp-transactions branch from 4580bc2 to 03853cc Compare

July 7, 2025 14:14

chirag-parmar requested a review from bhartnett

July 8, 2025 05:26

chirag-parmar force-pushed the vp-transactions branch from 0c90a46 to 5e3712b Compare

July 8, 2025 09:02

Contributor

bhartnett commented Jul 9, 2025

It would be good to see some tests added to this PR.

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/rpc_eth_api.nim Show resolved Hide resolved

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

bhartnett reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Show resolved Hide resolved

tersec reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

tersec reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

tersec reviewed

View reviewed changes

nimbus_verified_proxy/rpc/receipts.nim Outdated Show resolved Hide resolved

chirag-parmar force-pushed the vp-transactions branch from 8003f3e to a026d61 Compare

July 17, 2025 08:33

chirag-parmar and others added 13 commits

July 28, 2025 09:20


          add block tests for forks

47159e9


          block walk test and api test

460fea4


          equality

f4703c3


          remove unwanted file

1e1c851


          txs, receipts and logs

35567bf


          format

cabf932


          add copyright

8b0952a


          Apply suggestions from code review

637c3c0

Co-authored-by: bhartnett <[email protected]>


          review and fixes

c88910b


          add methods to backend

8bb05d9


          fixes

4dd44f4


          fixes

8e9ff3a


          Merge branch 'vp-tests' into vp-transactions

586a705

chirag-parmar added 2 commits

July 28, 2025 22:09


          tx tests

1e89442


          receipts and logs tests

309be89

chirag-parmar force-pushed the vp-transactions branch from ed94308 to 309be89 Compare

July 31, 2025 10:33

chirag-parmar requested a review from bhartnett

July 31, 2025 11:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet