From ffc71ff7d2c27d419bff4ca127ed5219bf9261c3 Mon Sep 17 00:00:00 2001
From: kumavis <aaron@kumavis.me>
Date: Thu, 5 Apr 2018 13:38:34 -0700
Subject: migrator - dont overwrite error stack and warn to console

---
 app/scripts/lib/migrator/index.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'app/scripts/lib/migrator')

diff --git a/app/scripts/lib/migrator/index.js b/app/scripts/lib/migrator/index.js
index ea9af3c80..85c2717ea 100644
--- a/app/scripts/lib/migrator/index.js
+++ b/app/scripts/lib/migrator/index.js
@@ -29,9 +29,12 @@ class Migrator extends EventEmitter {
         // accept the migration as good
         versionedData = migratedData
       } catch (err) {
+        // rewrite error message to add context without clobbering stack
+        const originalErrorMessage = err.message
+        err.message = `MetaMask Migration Error #${migration.version}: ${originalErrorMessage}`
+        console.warn(err.stack)
         // emit error instead of throw so as to not break the run (gracefully fail)
-        const error = new Error(`MetaMask Migration Error #${migration.version}:\n${err.stack}`)
-        this.emit('error', error)
+        this.emit('error', err)
         // stop migrating and use state as is
         return versionedData
       }
-- 
cgit