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

Create an RPC endpoint for DataCommitment #620

Closed
evan-forbes opened this issue Jan 28, 2022 · 1 comment · Fixed by #625
Closed

Create an RPC endpoint for DataCommitment #620

evan-forbes opened this issue Jan 28, 2022 · 1 comment · Fixed by #625

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Jan 28, 2022

As described in #42, we will need to collect the data roots over a provided range of blocks, and then create a new merkle root of those data roots.

While this could be done by the orchestrator querying each block and extracting the data hash, it will likely be easier to simply perform the extraction in the celestia-core node, and serve this over an rpc endpoint.

@evan-forbes evan-forbes moved this to TODO in Celestia Node Jan 28, 2022
@evan-forbes evan-forbes changed the title Create an RPC endpoint to create a root of data hashes over a range of blocks Create an RPC endpoint for DataCommitment Jan 28, 2022
@rach-id rach-id removed the status in Celestia Node Feb 1, 2022
@rach-id rach-id moved this to In Progress in Celestia Node Feb 1, 2022
@adlerjohn adlerjohn linked a pull request Feb 3, 2022 that will close this issue
@rach-id rach-id moved this from In Progress to Done in Celestia Node Feb 5, 2022
@evan-forbes
Copy link
Member Author

closed with #625

evan-forbes pushed a commit that referenced this issue Jun 9, 2023
* Minor edit to RPC block search test

* Applied @sergio-mena s comments

(cherry picked from commit 9b9bd1a)

Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants