fix: use the feesReceiver to estimate native payment (#94)

* fix: use the feesReceiver to estimate native payment * build: update version
rsksmart · Jul 31, 2024 · a856fdd · a856fdd
1 parent f675abd
commit a856fdd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@rsksmart/rif-relay-client",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "private": false,
   "description": "This project contains all the client code for the rif relay system.",
   "license": "MIT",

diff --git a/src/utils.ts b/src/utils.ts
@@ -115,8 +115,13 @@ const estimatePaymentGas = async ({
   const isNativePayment = (await tokenContract) === constants.AddressZero;
 
   if (isNativePayment) {
+    // Ideally we should set `from` with the tokenOrigin address,
+    // but the tokenOrigin may not have funds, hence the estimation will fail
+    // so we set the feesReceiver as the `from` address.
+    // This may be wrong in case the feesReceiver performs some operations using
+    // the msg.sender address in the fallback/receive function.
     return await estimateInternalCallGas({
-      from: tokenOrigin,
+      from: feesReceiver,
       to: feesReceiver,
       gasPrice,
       value: tokenAmount,