Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Implement eth_getProof RPC message #382

Closed
facuspagnuolo opened this issue Mar 14, 2019 · 3 comments · Fixed by #3199
Closed

Implement eth_getProof RPC message #382

facuspagnuolo opened this issue Mar 14, 2019 · 3 comments · Fixed by #3199

Comments

@facuspagnuolo
Copy link

Would be really amazing to provide support for the eth_getProof RPC message. I actually needed to migrate bunch of tests to geth to make them work :)

Please let me know if I can help with it.

@davidmurdoch
Copy link
Member

Thanks for the feature request, @facuspagnuolo! I'd like to start implementing more of the DRAFT status EIPs into ganache-core sooner than we have been. So if you'd like to put in the effort to make this happen I'd happily review and merge (likely behind a new option/flag we've been wanting to implement: enable-eips: Array<string> (we're still working on the name).

The bulk of the code will need to go in https://github.com/trufflesuite/ganache-core/blob/develop/lib/blockchain_double.js

Let me know if you need any help with the code base!

@simon-jentzsch
Copy link

This would be great, even though I don't have much time right now, I would love to help ( I did the implementation for geth and parity ). So if I can help, just ask!

@saurik
Copy link

saurik commented Sep 28, 2020

Related: #725

@davidmurdoch davidmurdoch added this to the 7.1.0 milestone Jan 5, 2022
@MicaiahReid MicaiahReid changed the title Implement eth_getProof RPC message Implement eth_getProof RPC message Apr 22, 2022
@jeffsmale90 jeffsmale90 linked a pull request Jun 8, 2022 that will close this issue
@davidmurdoch davidmurdoch moved this to Inbox in Ganache Jul 19, 2022
@davidmurdoch davidmurdoch moved this from Inbox to Backlog in Ganache Jul 19, 2022
@davidmurdoch davidmurdoch moved this from Backlog to In Progress in Ganache Jul 19, 2022
Repository owner moved this from In Progress to Done in Ganache Jul 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants