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.

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

feat: add get prover data api function #18

Merged
merged 5 commits into from
Jun 22, 2023
Merged

feat: add get prover data api function #18

merged 5 commits into from
Jun 22, 2023

Conversation

atanmarko
Copy link
Member

@atanmarko atanmarko commented Jun 5, 2023 •
edited
Loading

Description

Implement jsonrpc eth_getProverData api function that returns account data used in the block.

Fixes TP-612

Additions and Changes

  • JSONRPC interface expanded with eth_getProverData function
  • Added prover module for implementation of data retrieval and filtering functions
  • Internal APIs are updated to run the tracer, retrieve merkle proof for the contract storage and state

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@atanmarko atanmarko self-assigned this Jun 5, 2023
@atanmarko atanmarko force-pushed the fix/jsonrpc-tracer branch 2 times, most recently from 3368ff6 to 5a51279 Compare June 6, 2023 09:16
@atanmarko atanmarko force-pushed the feat/add-prover-data branch 2 times, most recently from 001f80d to 70c6fb6 Compare June 6, 2023 12:19
@atanmarko atanmarko marked this pull request as ready for review June 6, 2023 12:20
@atanmarko atanmarko requested a review from a team as a code owner June 6, 2023 12:20
@atanmarko atanmarko requested review from hadjiszs and JDawg287 and removed request for a team June 6, 2023 12:20
@atanmarko atanmarko force-pushed the feat/add-prover-data branch 3 times, most recently from b214a86 to 575c391 Compare June 16, 2023 13:58
Base automatically changed from fix/jsonrpc-tracer to develop June 22, 2023 09:35
hadjiszs
hadjiszs approved these changes Jun 22, 2023
View reviewed changes
state/immutable-trie/snapshot.go Show resolved Hide resolved
state/immutable-trie/snapshot.go Show resolved Hide resolved
server/server.go Show resolved Hide resolved
@atanmarko atanmarko merged commit 74441b2 into develop Jun 22, 2023
6 of 7 checks passed
@atanmarko atanmarko deleted the feat/add-prover-data branch June 22, 2023 15:59
atanmarko added a commit that referenced this pull request Aug 18, 2023
@atanmarko
feat: add get prover data api function (#18)
358a069
atanmarko added a commit that referenced this pull request Sep 7, 2023
@atanmarko
feat: add get prover data api function (#18)
99814a7
atanmarko added a commit that referenced this pull request Sep 15, 2023
@atanmarko
feat: add get prover data api function (#18)
f28c822
atanmarko added a commit that referenced this pull request Oct 9, 2023
@atanmarko
feat: add get prover data api function (#18)
49a887b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hadjiszs hadjiszs hadjiszs approved these changes

@JDawg287 JDawg287 Awaiting requested review from JDawg287 JDawg287 is a code owner automatically assigned from topos-protocol/protocol

Assignees

@atanmarko atanmarko

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@atanmarko @hadjiszs