Merge pull request #1178 from blorgon1/fix/validator-slashing-update

fix: decoding validator update from slashing

CHANGELOG.md

@@ -17,8 +17,10 @@ and this project adheres to
 
 - @cosmjs/stargate: Fix valid values of `BondStatusString` for `validators`
   query ([#1170]).
+- @cosmjs/tendermint-rpc: Fix decoding validator updates due to slashing ([#1177]).
 
 [#1170]: https://github.com/cosmos/cosmjs/issues/1170
+[#1177]: https://github.com/cosmos/cosmjs/issues/1177
 
 ### Changed
 

packages/tendermint-rpc/src/tendermint34/adaptor/responses.spec.ts

@@ -49,6 +49,27 @@ describe("Adaptor Responses", () => {
         votingPower: 11418237,
       });
     });
+
+    it("works for block results format without voting power", () => {
+      // from https://rpc.cosmos.network/block_results?height=10883046
+      const update = decodeValidatorUpdate({
+        pub_key: {
+          Sum: {
+            type: "tendermint.crypto.PublicKey_Ed25519",
+            value: {
+              ed25519: "HjSC7VkhKih6xMhudlqfaFE8ZZnP8RKJPv4iqR7RhcE=",
+            },
+          },
+        },
+      });
+      expect(update).toEqual({
+        pubkey: {
+          algorithm: "ed25519",
+          data: fromBase64("HjSC7VkhKih6xMhudlqfaFE8ZZnP8RKJPv4iqR7RhcE="),
+        },
+        votingPower: 0,
+      });
+    });
   });
 
   describe("decodeValidatorInfo", () => {

packages/tendermint-rpc/src/tendermint34/adaptor/responses.ts

@@ -257,13 +257,14 @@ function decodeConsensusParams(data: RpcConsensusParams): responses.ConsensusPar
 // for block results
 interface RpcValidatorUpdate {
   readonly pub_key: RpcPubkey;
-  readonly power: string;
+  // When omitted, this means zero (see https://github.com/cosmos/cosmjs/issues/1177#issuecomment-1160115080)
+  readonly power?: string;
 }
 
 export function decodeValidatorUpdate(data: RpcValidatorUpdate): responses.ValidatorUpdate {
   return {
     pubkey: decodePubkey(assertObject(data.pub_key)),
-    votingPower: Integer.parse(assertNotEmpty(data.power)),
+    votingPower: Integer.parse(data.power ?? 0),
   };
 }
 

packages/tendermint-rpc/src/tendermint35/adaptor/responses.spec.ts

@@ -49,6 +49,27 @@ describe("Adaptor Responses", () => {
         votingPower: 11418237,
       });
     });
+
+    it("works for block results format without voting power", () => {
+      // from https://rpc.cosmos.network/block_results?height=10883046
+      const update = decodeValidatorUpdate({
+        pub_key: {
+          Sum: {
+            type: "tendermint.crypto.PublicKey_Ed25519",
+            value: {
+              ed25519: "HjSC7VkhKih6xMhudlqfaFE8ZZnP8RKJPv4iqR7RhcE=",
+            },
+          },
+        },
+      });
+      expect(update).toEqual({
+        pubkey: {
+          algorithm: "ed25519",
+          data: fromBase64("HjSC7VkhKih6xMhudlqfaFE8ZZnP8RKJPv4iqR7RhcE="),
+        },
+        votingPower: 0,
+      });
+    });
   });
 
   describe("decodeValidatorInfo", () => {

packages/tendermint-rpc/src/tendermint35/adaptor/responses.ts

@@ -259,13 +259,14 @@ function decodeConsensusParams(data: RpcConsensusParams): responses.ConsensusPar
 // for block results
 interface RpcValidatorUpdate {
   readonly pub_key: RpcPubkey;
-  readonly power: string;
+  // When omitted, this means zero (see https://github.com/cosmos/cosmjs/issues/1177#issuecomment-1160115080)
+  readonly power?: string;
 }
 
 export function decodeValidatorUpdate(data: RpcValidatorUpdate): responses.ValidatorUpdate {
   return {
     pubkey: decodePubkey(assertObject(data.pub_key)),
-    votingPower: Integer.parse(assertNotEmpty(data.power)),
+    votingPower: Integer.parse(data.power ?? 0),
   };
 }