aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfrankiebee <frankie.diamond@gmail.com>2017-08-24 13:48:19 +0800
committerfrankiebee <frankie.diamond@gmail.com>2017-08-24 13:50:40 +0800
commitdadee1ed79715c7dcdfe1d208e2bd554a1d4da8b (patch)
treee3a5f181833188b5c89594087eb2753e2aef5612
parent4de977e63e17e7a3804cb2fbc8674212838a8571 (diff)
downloadtangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar.gz
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar.bz2
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar.lz
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar.xz
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.tar.zst
tangerine-wallet-browser-dadee1ed79715c7dcdfe1d208e2bd554a1d4da8b.zip
hotfix - fail submitted txs whos nonce is out of bound
-rw-r--r--app/scripts/migrations/019.js60
-rw-r--r--app/scripts/migrations/index.js1
2 files changed, 61 insertions, 0 deletions
diff --git a/app/scripts/migrations/019.js b/app/scripts/migrations/019.js
new file mode 100644
index 000000000..e6e8f1f01
--- /dev/null
+++ b/app/scripts/migrations/019.js
@@ -0,0 +1,60 @@
+
+const version = 19
+
+/*
+
+This migration sets transactions with the 'Gave up submitting tx.' err message
+to a 'failed' stated
+
+*/
+
+const clone = require('clone')
+
+module.exports = {
+ version,
+
+ migrate: function (originalVersionedData) {
+ const versionedData = clone(originalVersionedData)
+ versionedData.meta.version = version
+ try {
+ const state = versionedData.data
+ const newState = transformState(state)
+ versionedData.data = newState
+ } catch (err) {
+ console.warn(`MetaMask Migration #${version}` + err.stack)
+ }
+ return Promise.resolve(versionedData)
+ },
+}
+
+function transformState (state) {
+ const newState = state
+ const transactions = newState.TransactionController.transactions
+ newState.TransactionController.transactions = transactions.map((txMeta, _, txList) => {
+ if (txMeta.status !== 'submitted') return txMeta
+
+ const confirmedTxs = txList.filter((tx) => tx.status === 'confirmed')
+ .filter((tx) => tx.txParams.from === txMeta.txParams.from)
+ .filter((tx) => tx.metamaskNetworkId.from === txMeta.metamaskNetworkId.from)
+ const highestConfirmedNonce = getHighestNonce(confirmedTxs)
+
+ if (parseInt(txMeta.txParams.nonce, 16) > highestConfirmedNonce + 1) {
+ txMeta.status = 'failed'
+ txMeta.err = {
+ message: 'nonce too high',
+ note: 'migration 019 custom error',
+ }
+ }
+ return txMeta
+ })
+ return newState
+}
+
+function getHighestNonce (txList) {
+ const nonces = txList.map((txMeta) => {
+ const nonce = txMeta.txParams.nonce
+ return parseInt(nonce || '0x0', 16)
+ })
+ const highestNonce = Math.max.apply(null, nonces)
+ return highestNonce
+}
diff --git a/app/scripts/migrations/index.js b/app/scripts/migrations/index.js
index 6bfc622f7..e9cbd7b98 100644
--- a/app/scripts/migrations/index.js
+++ b/app/scripts/migrations/index.js
@@ -29,4 +29,5 @@ module.exports = [
require('./016'),
require('./017'),
require('./018'),
+ require('./019'),
]