Require secp256k1 pubkey registration for unjail

[hacheigriega]

Explanation of Changes

We override the slashing module's unjail transaction to add an additional check for registration of required SEDA public keys.

Testing

Tested on a 4-node local network where one of the nodes was missing the required public key. When the proving scheme was activated, the node was jailed. Subsequently, its unjail attempts were blocked until it registered the required public key.

sedad tx slashing unjail --from validator4 --keyring-backend test --home $HOME/.sedad/validator4 --keyring-dir $HOME/.sedad/validator4 --gas-prices 10000000000aseda --gas auto --gas-adjustment 2.0 --keyring-backend test --chain-id testing --node http://localhost:26657 --yes

Error: rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: validator has not registered required SEDA keys: invalid request [cosmossdk.io/errors@v1.0.1/errors.go:151] with gas used: '33552': unknown request
10:47AM ERR failure when running app err="rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: validator has not registered required SEDA keys: invalid request [cosmossdk.io/errors@v1.0.1/errors.go:151] with gas used: '33552': unknown request"

Note there is a bug where the keyring directory doesn't default to the home directory given by the --home flag, so the --keyring-dir flag has to be provided. Created issue #435

Related PRs and Issues

Closes: #422