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

The section about light clients should be completely removed #660

Open
tomaka opened this issue Jun 15, 2023 · 0 comments
Open

The section about light clients should be completely removed #660

tomaka opened this issue Jun 15, 2023 · 0 comments

Comments

@tomaka
Copy link
Collaborator

tomaka commented Jun 15, 2023

It is not the role of the specification to define what a light client is. Anyone reading the spec can decide how they want to interact with the network, provided they meet the requirements found in the spec.
In other words, if you don't want your node to store the storage of the blocks, then just write the code that doesn't store the storage of the blocks. There's no need for the specification to explain that you're allowed to not do so.

Furthermore, these paragraphs are completely off-topic:

The client MUST support the following RPC methods: rpc_methods, chainHead_unstable_follow, chainHead_unstable_unfollow, chainHead_unstable_unpin, chainHead_unstable_storage, chainHead_unstable_call chainHead_unstable_stopCall. transaction_unstable_submitAndWatch, and transaction_unstable_unwatch

The client MUST support the @substrate/connect connection extension protocol: ToApplicationError, ToApplicationChainReady, ToApplicationRpc, ToExtensionAddChain, ToExtensionAddWellKnownChain, ToExtensionRpc, ToExtensionRemoveChain.

This specification is specifically about the interaction between peer-to-peer nodes.

And when it comes to section 7.4 about networking messages, it should be moved to the section about the networking. These messages are just as important for full nodes as they are for light clients, given that full nodes have to implement the code that answers these messages.

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

1 participant