Merge branch '129-fix-evm-estimate-gas' into 'dev'

fix evm gas estimation Closes #129 See merge request ergo/rosen-bridge/rosen-chains!160
rosen-bridge · Dec 16, 2024 · 6b95cef · 6b95cef
2 parents 1767994 + f409b2a
commit 6b95cef
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/soft-ads-agree.md b/.changeset/soft-ads-agree.md
@@ -0,0 +1,5 @@
+---
+'@rosen-chains/evm-rpc': patch
+---
+
+Fix EVM gas estimation so that only required fields are sent to the RPC
diff --git a/packages/networks/evm-rpc/lib/EvmRpcNetwork.ts b/packages/networks/evm-rpc/lib/EvmRpcNetwork.ts
@@ -300,8 +300,9 @@ class EvmRpcNetwork extends AbstractEvmNetwork {
   getGasRequired = async (transaction: Transaction): Promise<bigint> => {
     try {
       const gas = await this.provider.estimateGas({
-        ...transaction.toJSON(),
         from: this.lockAddress,
+        to: transaction.to,
+        data: transaction.data,
       });
       this.logger.debug(
         `requested 'estimateGas' of ${