Merge PR #5618: Add test of missing verifier in cli context

This commit is contained in:
Anthony
2020-02-06 10:27:23 -05:00
committed by GitHub
parent a7f25ade03
commit b0c6c750df
2 changed files with 4 additions and 0 deletions
+1
View File
@@ -50,6 +50,7 @@ balances or a single balance by denom when the `denom` query parameter is presen
### Bug Fixes
* (x/bank) [\#5531](https://github.com/cosmos/cosmos-sdk/issues/5531) Added missing amount event to MsgMultiSend, emitted for each output.
* (client) [\#5618](https://github.com/cosmos/cosmos-sdk/pull/5618) Fix crash on the client when the verifier is not set.
### State Machine Breaking
+3
View File
@@ -130,6 +130,9 @@ func (ctx CLIContext) query(path string, key tmbytes.HexBytes) ([]byte, int64, e
// Verify verifies the consensus proof at given height.
func (ctx CLIContext) Verify(height int64) (tmtypes.SignedHeader, error) {
if ctx.Verifier == nil {
return tmtypes.SignedHeader{}, fmt.Errorf("missing valid certifier to verify data from distrusted node")
}
check, err := tmliteProxy.GetCertifiedCommit(height, ctx.Client, ctx.Verifier)
switch {
case tmliteErr.IsErrCommitNotFound(err):