lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #1137 from chaotixkilla/fix-erc20-transfers

Browse source 
Fix ERC-20 transfer gas estimation and receiver address
This commit is contained in:
Rafael Taranto 2022-03-03 15:06:38 +00:00 • committed by GitHub
commit e3a1bb8f47
1 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
lib/plugins/wallet/geth/base.js
View file

@ -102,12 +102,20 @@ function generateTx (_toAddress, wallet, amount, includesFee, cryptoCode) {
const isErc20Token = coins.utils.isErc20Token(cryptoCode) const isErc20Token = coins.utils.isErc20Token(cryptoCode)
const toAddress = isErc20Token ? coins.utils.getErc20Token(cryptoCode).contractAddress : _toAddress.toLowerCase() const toAddress = isErc20Token ? coins.utils.getErc20Token(cryptoCode).contractAddress : _toAddress.toLowerCase()
let contract, contractData
if (isErc20Token) {
contract = web3.eth.contract(ABI.ERC20).at(toAddress)
contractData = isErc20Token && contract.transfer.getData(_toAddress.toLowerCase(), hex(toSend))
}
const txTemplate = { const txTemplate = {
from: fromAddress, from: fromAddress,
to: toAddress, to: toAddress,
value: amount.toString() value: amount.toString()
} }
if (isErc20Token) txTemplate.data = contractData
const promises = [ const promises = [
pify(web3.eth.estimateGas)(txTemplate), pify(web3.eth.estimateGas)(txTemplate),
pify(web3.eth.getGasPrice)(), pify(web3.eth.getGasPrice)(),
@ -138,9 +146,7 @@ function generateTx (_toAddress, wallet, amount, includesFee, cryptoCode) {
} }
if (isErc20Token) { if (isErc20Token) {
const contract = web3.eth.contract(ABI.ERC20).at(toAddress) rawTx.data = contractData
if (contract)
rawTx.data = contract.transfer.getData(toAddress, hex(toSend))
} }
const tx = new Tx(rawTx) const tx = new Tx(rawTx)