dymensionxyz / dymension-rdk

Framework for building highly scalable RollApps
Apache License 2.0
98 stars 54 forks source link

Use `AccAddress` instead of `ValAddress` for operator address #592

Closed keruch closed 3 weeks ago

keruch commented 3 weeks ago

Use AccAddress instead of ValAddress for the operator address in x/sequencer messages.

keruch commented 3 weeks ago

We used the sequencer operator address as ValAddress in RDK historically, but in the Hub the sequencer operator address is AccAddress. Now, after relayer and whitelisted addr refactoring, we use the Hub as a source of truth, so address formats are jumbled.

I cleaned it up in the following way:

So operator address:

It also applies to MsgUpdateWhitelistedRelayers and MsgUpdateRewardAddress. In the Hub the signer is AccAddress, in RDK ValAddress, though in theory in RDK these messages will be unused.