Skip to content

Commit

Permalink
Merge pull request #310 from hyperlane-xyz/trevor/bump-cosmos-gas-est…
Browse files Browse the repository at this point in the history
…imate-mult

chore: increase the Cosmos gas estimate multiplier
  • Loading branch information
tkporter authored Nov 6, 2024
2 parents 99d8bf0 + 13992ac commit 0a7891a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/features/wallet/hooks/cosmos.ts
Original file line number Diff line number Diff line change
Expand Up @@ -108,8 +108,8 @@ export function useCosmosTransactionFns(): ChainTransactionFns {
// The fee param of 'auto' here stopped working for Neutron-based IBC transfers
// It seems the signAndBroadcast method uses a default fee multiplier of 1.4
// https://github.com/cosmos/cosmjs/blob/e819a1fc0e99a3e5320d8d6667a08d3b92e5e836/packages/stargate/src/signingstargateclient.ts#L115
// Bumping to 1.6 fixes the insufficient gas issue
result = await client.signAndBroadcast(chainContext.address, [tx.transaction], 1.6);
// A multiplier of 1.6 was insufficient for Celestia -> Neutron|Cosmos -> XXX transfers, but 2 worked.
result = await client.signAndBroadcast(chainContext.address, [tx.transaction], 2);
txDetails = await client.getTx(result.transactionHash);
} else {
throw new Error(`Invalid cosmos provider type ${tx.type}`);
Expand Down

0 comments on commit 0a7891a

Please sign in to comment.