search

penumbra-zone / web

Apache License 2.0
10 stars 7 forks source link

Implement complete confirmation of chain invariants #1264

Open turbocrime opened 3 weeks ago

turbocrime commented 3 weeks ago

currently, only chainId is verified. these other app parameters should be confirmed as well.

https://github.com/penumbra-zone/penumbra/blob/961d5f2127754d08996892bb39fbd3d956c74e30/crates/core/app/src/params/change.rs#L115-L150

may be affected by https://github.com/penumbra-zone/penumbra/issues/3593

verification happens in multiple locations, but should be centralized, or at least made re-useable.

verification should occur:

  1. at genesis init
  2. when connecting to a new rpc for the first time
  3. when starting up and reconnecting to the configured rpc
  4. when reading updated appParameters

other aspects of chain governance are out of scope