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

spec: proof of fork #461

Closed
ebuchman opened this issue Jul 19, 2020 · 3 comments
Closed

spec: proof of fork #461

ebuchman opened this issue Jul 19, 2020 · 3 comments
Assignees
Labels
spec Specifications

Comments

@ebuchman
Copy link
Member

Draft of the problem with more of an IBC lens is here.

Primary next step is to flesh out the "proof of fork" data structures and protocol for light clients to submit to full nodes and to on-chain handlers.

See the next steps in the doc and further discussion in the first PR reviews

@ebuchman
Copy link
Member Author

As far as relationship between this follow up and the Go code:

  1. "proof of fork" for on-chain clients - may require breaking changes to ICS07
  2. "proof of fork" for full nodes - non-breaking changes to Tendermint Core
  3. "evidence of misbehaviour" - breaking changes to Tendermint Core

Thus we should prioritize (1) and (3) as these may be breaking on IBC and Tendermint respectively, while (2) would be less disruptive changes to the RPC. That said, of course figuring out (2) may impact (3), so they need to be somewhat thought of together.

Since these changes are to a generally underused component of the system (evidence handling), these changes should not block other integrations against Stargate releases, but they may require testnet restarts.

@ebuchman
Copy link
Member Author

@milosevic @josef-widder should we close this?

@josef-widder
Copy link
Member

Yes. The on-chain accountability protocol is the next thing we are doing, but there are other issues for that.

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

No branches or pull requests

3 participants