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

Initial evidence types #362

Closed
sync-by-unito bot opened this issue May 27, 2021 · 1 comment
Closed

Initial evidence types #362

sync-by-unito bot opened this issue May 27, 2021 · 1 comment

Comments

@sync-by-unito
Copy link

sync-by-unito bot commented May 27, 2021

Currently the only evidence type is double-signing. There should be additional types for anything slashable, i.e. anything that would trigger a fraud proof.

Upcoming new Tendermint evidence types: https://github.com/tendermint/tendermint/blob/proto-breakage/proto/types/evidence.proto

Also see: https://medium.com/tendermint/different-types-of-evidence-in-tendermint-5de4440fdd54

┆Issue is synchronized with this Asana task by Unito

@sync-by-unito
Copy link
Author

sync-by-unito bot commented May 27, 2021

➤ Ismail Khoffi commented:

Additionally to defining the exact data structures of different fraud proofs / evidence types, the specification should also stipulate who (which node types) generates them under which circumstances and what happens with the evidence after construction (does get included in the next block, does it trigger a slashing event, is it "only" gossipped in the p2p network - if yes to which nodes) and why it is generated in the first place.
This is to actually guide the implementation.

In the context of erasure coding, for instance, this is needed to inform the APIs of the used libraries (e.g. see celestiaorg/rsmt2d#24 and github.com/celestiaorg/rsmt2d#12).

Ideally, the spec also defined properties or invariants (if any) that are achieved by dispersing the evidence (e.g. a proposer can not propose an incorrectly encoded block without getting caught and getting slashed for that, etc).

@liamsi liamsi closed this as completed May 27, 2021
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