Merge pull request #7295 from LedgerHQ/fix/release-zondax-sign-tx

Release: Fix sign tx for injective
LedgerHQ · Jul 9, 2024 · 20ff796 · 20ff796
2 parents 6b6f36d + c59b06f
commit 20ff796
Show file tree

Hide file tree

Showing 5 changed files with 1,387 additions and 11 deletions.
diff --git a/.changeset/gentle-maps-juggle.md b/.changeset/gentle-maps-juggle.md
@@ -0,0 +1,5 @@
+---
+"@ledgerhq/live-common": minor
+---
+
+Fix Cosmos library to sign tx
diff --git a/.pnpmfile.cjs b/.pnpmfile.cjs
@@ -85,6 +85,10 @@ function readPackage(pkg, context) {
       addDependencies("@cosmjs/tendermint-rpc", {
         "@cosmjs/utils": pkg.version,
       }),
+
+      removeDependencies("@zondax/ledger-cosmos-js", ["crypto"], {
+        kind: "dependencies",
+      }),
       /* React Native and Metro bundler packages */
       // Crashes ios build if removed /!\
       addDependencies("@react-native/codegen", {

diff --git a/libs/ledger-live-common/package.json b/libs/ledger-live-common/package.json
@@ -196,7 +196,7 @@
     "@xstate/react": "^1.6.3",
     "@zondax/cbor": "v8.1.0-zondax-no-bigint",
     "@zondax/ledger-casper": "^2.6.1",
-    "@zondax/ledger-cosmos-js": "3.0.2",
+    "@zondax/ledger-cosmos-js": "3.0.3",
     "@zondax/ledger-filecoin": "^0.11.2",
     "@zondax/ledger-icp": "^0.7.0",
     "@zondax/ledger-stacks": "^1.0.2",

diff --git a/libs/ledger-live-common/src/families/cosmos/signOperation.ts b/libs/ledger-live-common/src/families/cosmos/signOperation.ts
@@ -66,7 +66,7 @@ export const signOperation: AccountBridge<Transaction>["signOperation"] = ({
           // HRP is only needed when signing for ethermint chains
           const signResponseApp =
             path[1] === 60
-              ? await app.sign(path, tx, parseInt(chainInstance.prefix))
+              ? await app.sign(path, tx, chainInstance.prefix)
               : await app.sign(path, tx);
 
           switch (signResponseApp.return_code) {