[go: up one dir, main page]

Proto,tx_rollup: Add lazy slashing for rejected commitment’s successors

Context

See commit description.

Prior to this patch, only the committers of rejected commitments see
their frozen bond burn.

Commitments that are descendants of rejected commitments are simply
overwritten.  As a consequence, committers of these commitments can
never reclaim their bond.  This is probably safe enough, but there is
a trick: frozen bonds are still part of their stake.

This patch changes that.  Now, prior to updating the layer-1 storage
with a commitment, we check whether one exists already or not.  If so,
we burn their bond once and for all.  As a safety net to avoid edge
cases, we forbid an implicit account to overwrite one of its own
commitment.

Manually testing the MR

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Thomas Letan

Merge request reports

Loading