celestiaorg / celestia-app

PoS application for the consensus portion of the Celestia network. Built using celestia-core (fork of CometBFT) and the cosmos-sdk
https://celestia.org
Apache License 2.0
328 stars 261 forks source link

Questions about MsgExec inside MsgExec #3609

Open rootulp opened 1 week ago

rootulp commented 1 week ago

Context

https://github.com/celestiaorg/celestia-app/pull/3555#discussion_r1635492368 https://github.com/celestiaorg/celestia-app/issues/3586

Problem

It is possible to nest an authz MsgExec message inside another authz MsgExec message.

Questions

  1. Is this actually problematic?
  2. Are there valid use cases to nest a MsgExec inside another MsgExec?
  3. How many layers of nesting do valid use cases use?
  4. Should we reject transactions (i.e. via a new antehandler) that have multiple layers of nesting?