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

rpc: revisit block and other RPC endpoints #195

Closed
liamsi opened this issue Mar 9, 2021 · 1 comment
Closed

rpc: revisit block and other RPC endpoints #195

liamsi opened this issue Mar 9, 2021 · 1 comment
Labels
C:rpc Component: RPC

Comments

@liamsi
Copy link
Member

liamsi commented Mar 9, 2021

Tendermint provides a bunch of rpc endpoints which are partly used in the SDK too. They are also used by block explorers. We should revisit each endpoint from the lense of LL and need to modify or maybe even ditch some.

Particularly the Block endpoint (ref: #189) should probably just return the state transitioning Tx and not the whole Block.Data.

Nodes should request the Block.Data from ipfs / the p2p network like everyone else. To make it as easy as possible for block explorers and other existing tooling in the Tendermint/Cosmos ecosystem to also become LazyLedger block explorers, it would still be good to keep an rpc endpoint, that serves the Block minus the application specific blobs (messages). If that endpoint is still called Block and only serves the state-modifying portion, block explorers could easily switch.

ref: #189

Some examples of block explorers:
https://cosmos.bigdipper.live/blocks/5434576
https://www.mintscan.io/cosmos/blocks

@liamsi liamsi added the C:rpc Component: RPC label Mar 9, 2021
@liamsi liamsi changed the title rpc: revisit block and other endpoints rpc: revisit block and other RPC endpoints Jun 14, 2021
@liamsi
Copy link
Member Author

liamsi commented Aug 16, 2021

closing this for now. If anything we now want to extend the RPC and not limit it.

@liamsi liamsi closed this as completed Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:rpc Component: RPC
Projects
None yet
Development

No branches or pull requests

1 participant