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: implement RPC Discover dynamically #124

Open
meowsbits opened this issue Jan 23, 2020 · 0 comments
Open

feat: implement RPC Discover dynamically #124

meowsbits opened this issue Jan 23, 2020 · 0 comments

Comments

@meowsbits
Copy link
Member

Currently the rpc.discover API method returns a static string, fitting the spec documented here: https://github.com/etclabscore/ethereum-json-rpc-specification/blob/master/openrpc.json.

This spec represents a minimum sufficient interface that clients should support.

However this spec does not limit the client from supporting further methods or documenting them via this method.

Instead of multigeth returning a static minimum-viable string at this endpoint, it would be really great to be able to have multigeth actually return a complete representation of the methods that it is making available, which are a lot.

"Handrolling" a specification object to this purpose costs technical debt in the short and long terms, is prone to error, and generally an inelegant way of solving the challenge.

Better, to devise a system that can be applied at the RPC server level, which can programmatically generate an OpenRPC schema fitting the actual API available. This would work, then, hand-in-hand with github.com/gregdhill/go-openrpc (that codebase can consume schemas but not generate them).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant