Hopefully fix gas limit issues and don't use 3.5M gas by default

DistributedCollective · May 29, 2024 · 07eb468 · 07eb468
1 parent 74306d7
commit 07eb468
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/sovryn-token-bridge/federator/src/lib/TransactionSender.js b/sovryn-token-bridge/federator/src/lib/TransactionSender.js
@@ -51,9 +51,14 @@ module.exports = class TransactionSender {
 
         // Vote: ~70k
         // Vote+execute: A little over 250k
+        // Vote+execute with signatures: Can get to over 300k, 500k+ with aggregators involved
         // First side token deployment: ~3.15M
-        const minimum = 300000;
-        const gasLimit = estimatedGas < minimum ? minimum : 3500000;
+        const multiplier = 1.5;
+        const minimum = 400000;
+        let gasLimit = Math.ceil(estimatedGas * multiplier);
+        if (gasLimit < minimum) {
+            gasLimit = minimum;
+        }
 
         return gasLimit;
     }