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

feat: add output root rpc method #86

Closed
ncitron opened this issue Apr 17, 2023 · 3 comments · Fixed by #102
Closed

feat: add output root rpc method #86

ncitron opened this issue Apr 17, 2023 · 3 comments · Fixed by #102
Assignees
Labels
M-easy Meta: Good first issue

Comments

@ncitron
Copy link
Contributor

ncitron commented Apr 17, 2023

The Optimism specs for the rollup client include a special RPC method called optimism_outputAtBlock which returns the L2 output root for a given block number. This is used by the proposer component to post the outputs to L1. While we're a long way away from connecting Magi to the proposer, we can pretty easily support this. Lets use the jsonrpsee crate for this.

More info on the RPC method here

@ncitron ncitron added the M-easy Meta: Good first issue label Apr 17, 2023
@shrimalmadhur
Copy link
Contributor

shrimalmadhur commented Apr 19, 2023

@ncitron I am down to give this a try. I am not that familiar with rust and just starting but definitely would be able to help. :) Let me know if there is any specific guidance for it apart from the crate you mentioned

@ncitron
Copy link
Contributor Author

ncitron commented Apr 19, 2023

That would be great. When you implement this, can we make a new top level module called rpc and put the logic there (then start it from the driver module).

@shrimalmadhur
Copy link
Contributor

Okay. on it. might have more questions later. but I am good to start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-easy Meta: Good first issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants