search

dymensionxyz / dymension

Dymension Hub
https://dymension.xyz
Other
362 stars 338 forks source link

x/sequencers: check jail not imply 100% slash assumption #1063

Open danwt opened 1 month ago

danwt commented 1 month ago

The bond reduction feature was coded assuming all jails and slashes result in 100% token loss, which is not true

Needs to be checked

https://github.com/dymensionxyz/dymension/pull/1031#pullrequestreview-2228215187

To clarify:

Slashes incurred from liveness fault, which are not also Jail, should slash param% of tokens. Slashes incurred by fraud should slash 100% of remaining tokens

danwt commented 3 weeks ago

~In product we decided it's ok~