feature/default_network_editable
brunobar79 6 years ago
parent 12b41b8fc2
commit 068bf43615
  1. 4
      app/scripts/controllers/transactions/index.js
  2. 12
      app/scripts/eth-ledger-keyring-listener.js

@ -289,10 +289,10 @@ class TransactionController extends EventEmitter {
// sign tx
const fromAddress = txParams.from
const ethTx = new Transaction(txParams)
await this.signEthTx(ethTx, fromAddress)
const signedTx = await this.signEthTx(ethTx, fromAddress)
// set state to signed
this.txStateManager.setTxStatusSigned(txMeta.id)
const rawTx = ethUtil.bufferToHex(ethTx.serialize())
const rawTx = ethUtil.bufferToHex(signedTx.serialize())
return rawTx
}

@ -32,9 +32,6 @@ class LedgerKeyring extends EventEmitter {
this.iframe.src = ORIGIN
console.log('Injecting ledger iframe')
document.head.appendChild(this.iframe)
console.log('[LEDGER]: LEDGER FROM-IFRAME LISTENER READY')
}
sendMessage (msg, cb) {
@ -181,8 +178,6 @@ class LedgerKeyring extends EventEmitter {
return new Promise((resolve, reject) => {
this.unlock()
.then(_ => {
console.log('[LEDGER]: sending message ', 'ledger-sign-transaction')
this.sendMessage({
action: 'ledger-sign-transaction',
params: {
@ -196,12 +191,11 @@ class LedgerKeyring extends EventEmitter {
gasLimit: this._normalize(tx.gasLimit),
gasPrice: this._normalize(tx.gasPrice),
},
path: this._pathFromAddress(address)
path: this._pathFromAddress(address),
},
},
({action, success, payload}) => {
if (success) {
console.log('[LEDGER]: got tx signed!', payload.txData)
const signedTx = new Transaction(payload.txData)
// Validate that the signature matches the right address
const addressSignedWith = ethUtil.toChecksumAddress(`0x${signedTx.from.toString('hex')}`)
@ -209,9 +203,6 @@ class LedgerKeyring extends EventEmitter {
if (addressSignedWith !== correctAddress) {
reject('signature doesnt match the right address')
}
console.log('[LEDGER]: all good!', signedTx.toJSON())
console.log('[LEDGER]: signedTX', `0x${signedTx.serialize().toString('hex')}`)
resolve(signedTx)
} else {
reject(payload)
@ -230,7 +221,6 @@ class LedgerKeyring extends EventEmitter {
return new Promise((resolve, reject) => {
this.unlock()
.then(_ => {
console.log('[LEDGER]: sending message ', 'ledger-sign-personal-message')
this.sendMessage({
action: 'ledger-sign-personal-message',
params: {

Loading…
Cancel
Save