safety check for WC requests (#780)

rainbow-me · Jun 15, 2020 · 8f634f6 · 8f634f6
1 parent 4190da0
commit 8f634f6
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/parsers/requests.js b/src/parsers/requests.js
@@ -28,6 +28,22 @@ export const getRequestDisplayDetails = (payload, assets, nativeCurrency) => {
     payload.method === SIGN_TRANSACTION
   ) {
     const transaction = get(payload, 'params[0]', null);
+
+    // Backwards compatibility with param name change
+    if (transaction.gas && !transaction.gasLimit) {
+      transaction.gasLimit = transaction.gas;
+    }
+
+    // We must pass a number through the bridge
+    if (!transaction.gasLimit) {
+      transaction.gasLimit = 0;
+    }
+
+    // Dapps usually won't send this
+    if (!transaction.nonce) {
+      transaction.nonce = 0;
+    }
+
     return getTransactionDisplayDetails(
       transaction,
       assets,