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

Up-to-date JSON-RPC API spec #2053

Closed
pedrouid opened this issue May 20, 2019 · 4 comments
Closed

Up-to-date JSON-RPC API spec #2053

pedrouid opened this issue May 20, 2019 · 4 comments

Comments

@pedrouid
Copy link
Contributor

There are have been multiple issues opened and closed around this issue but we currently lack an updated JSON-RPC API spec.

I'm happy with keeping the same Github Wiki we have right now for reference (https://github.com/ethereum/wiki/wiki/JSON-RPC) but it needs to include the latest methods that have already been implement by clients and other middleware. For example: eth_signTransaction and eth_chainId

PS - This issue has already been referenced by #728 over a year ago

@axic
Copy link
Member

axic commented May 22, 2019

Can you check https://eips.ethereum.org/EIPS/eip-1474 if that satisfies these requirements? Please note Jekyll fails to render it properly (trying to fix that in #2059).

@pedrouid
Copy link
Contributor Author

Can you check https://eips.ethereum.org/EIPS/eip-1474 if that satisfies these requirements? Please note Jekyll fails to render it properly (trying to fix that in #2059).

Oh I never noticed the methods were included in 1474 because of the rendering. We should get some Jekyll folks in here to fix it.

Also is eth_chainId (#695) not included because it's not final yet?

@axic
Copy link
Member

axic commented Jun 20, 2019

Also is eth_chainId (#695) not included because it's not final yet?

Not included where? In 1474?

I'd say 1474 should be just the core spec and others refer to it. Later we could consider checkpointing a combined updated version of 1474 under a new number, which folds all those into one spec, if there's need.

@axic
Copy link
Member

axic commented Jun 20, 2019

Closing this as the rendering issue is tracked in #1878.

@axic axic closed this as completed Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants