Merge branch 'TxManager' of https://github.com/MetaMask/metamask-plugin into TxManager

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Remove certain non-essential permissions from certain builds.
 - Add a check for when a tx is included in a block.
+- Implement replay attack protections allowed by EIP 155.
 
 ## 2.14.1 2016-12-20
 

app/scripts/metamask-controller.js

@@ -186,8 +186,8 @@ module.exports = class MetamaskController {
       // tx signing
       approveTransaction: this.newUnsignedTransaction.bind(this),
       signTransaction: (...args) => {
-        this.setupSigningListners(...args)
-        this.txManager.formatTxForSigining(...args)
+        this.setupSigningListeners(...args)
+        this.txManager.formatTxForSigning(...args)
         this.sendUpdate()
       },
 
@@ -266,7 +266,7 @@ module.exports = class MetamaskController {
     })
   }
 
-  setupSigningListners (txParams) {
+  setupSigningListeners (txParams) {
     var txId = txParams.metamaskId
     // apply event listeners for signing and formating events
     this.txManager.once(`${txId}:formatted`, this.keyringController.signTransaction.bind(this.keyringController))

app/scripts/transaction-manager.js

@@ -137,7 +137,12 @@ module.exports = class TransactionManager extends EventEmitter {
   }
 
   // formats txParams so the keyringController can sign it
-  formatTxForSigining (txParams, cb) {
+  formatTxForSigning (txParams, cb) {
+    this.getNetwork((err, networkId) => {
+      if (err) {
+        return cb(err)
+      }
+
       var address = txParams.from
       var metaTx = this.getTx(txParams.metamaskId)
       var gasMultiplier = metaTx.gasMultiplier
@@ -152,10 +157,13 @@ module.exports = class TransactionManager extends EventEmitter {
       txParams.data = normalize(txParams.data)
       txParams.gasLimit = normalize(txParams.gasLimit || txParams.gas)
       txParams.nonce = normalize(txParams.nonce)
+      txParams.chainId = parseInt(networkId)
+
       const ethTx = new Transaction(txParams)
 
       // listener is assigned in metamaskController
       this.emit(`${txParams.metamaskId}:formatted`, ethTx, address, txParams.metamaskId, cb)
+    })
   }
 
   // receives a signed tx object and updates the tx hash