7117056 · bridge · git.iko.soy

git.iko.soy/bridge

on master help sign in

Merge pull request #239 from urbit/mino-bignum-mvp

7117056Fang7 years ago

src/lib/reticket.js+11−10

compare with master

unifiedsplit

@@ -12,7 +12,7 @@ import { addHexPrefix } from './wallet';

12import { sendAndAwaitAll } from './txn';

13import getSuggestedGasPrice from './getSuggestedGasPrice';

14import { GAS_LIMITS } from './constants';

15−import { toWei } from 'web3-utils';

15+import { toWei, toBN } from 'web3-utils';

17// the initial network key revision is always 1

18const INITIAL_NETWORK_KEY_REVISION = 1;

@@ -143,14 +143,15 @@ export async function reticketPointBetweenWallets({

143 progress(TRANSACTION_PROGRESS.SIGNING);

144

145 const suggestedGasPrice = await getSuggestedGasPrice(networkType);

146− const gasPrice = parseInt(toWei(suggestedGasPrice.toFixed(), 'gwei'), 10);

146+ const gasPrice = toWei(suggestedGasPrice.toFixed(), 'gwei');

147+ const gasPriceBN = toBN(gasPrice);

147− let totalCost = 0;

148+ let totalCost = toBN(0);

148 let inviteNonce = await web3.eth.getTransactionCount(fromWallet.address);

149 let inviteNonce = await web3.eth.getTransactionCount(fromWallet.address);

149 txs = txs.map(tx => {

150 txs = txs.map(tx => {

150 tx.from = fromWallet.address;

151 tx.from = fromWallet.address;

151 tx.nonce = inviteNonce++;

152 tx.nonce = inviteNonce++;

152− tx.gasPrice = gasPrice;

153+ tx.gasPrice = gasPriceBN;

153− totalCost = totalCost + tx.gasPrice * tx.gas;

154+ totalCost = totalCost.add(gasPriceBN.mul(toBN(tx.gas)));

154 return tx;

155 return tx;

155 });

156 });

156

157

@@ -194,15 +195,15 @@ export async function reticketPointBetweenWallets({

194 progress(TRANSACTION_PROGRESS.CLEANING);

195 progress(TRANSACTION_PROGRESS.CLEANING);

195

196

196 // if non-trivial eth left in invite wallet, transfer to new ownership

197 // if non-trivial eth left in invite wallet, transfer to new ownership

197− let balance = await web3.eth.getBalance(fromWallet.address);

198+ let balance = toBN(await web3.eth.getBalance(fromWallet.address));

198 const gasLimit = GAS_LIMITS.SEND_ETH;

199 const gasLimit = GAS_LIMITS.SEND_ETH;

199− const sendEthCost = gasPrice * gasLimit;

200+ const sendEthCost = gasPriceBN.mul(toBN(gasLimit));

200− if (transferEth && balance > sendEthCost) {

201+ if (transferEth && balance.gt(sendEthCost)) {

201− const value = balance - sendEthCost;

202+ const value = balance.sub(sendEthCost);

202 const tx = {

203 const tx = {

203 to: toWallet.ownership.keys.address,

204 to: toWallet.ownership.keys.address,

204 value: value,

205 value: value,

205− gasPrice: gasPrice,

206+ gasPrice: gasPriceBN,

206 gas: gasLimit,

207 gas: gasLimit,

207 nonce: inviteNonce++,

208 nonce: inviteNonce++,

208 };

209 };

src/lib/tank.js+22−14

@@ -1,6 +1,7 @@

1// /lib/tank: functions for funding transactions

3import retry from 'async-retry';

4+import { toBN } from 'web3-utils';

4import { RETRY_OPTIONS, waitForTransactionConfirm } from './txn';

5import { RETRY_OPTIONS, waitForTransactionConfirm } from './txn';

6//NOTE if accessing this in a localhost configuration fails with "CORS request

7//NOTE if accessing this in a localhost configuration fails with "CORS request

@@ -53,9 +54,10 @@ const ensureFundsFor = async (

53 askForFunding,

54 askForFunding,

54 gotFunding

55 gotFunding

55) => {

56) => {

56− let balance = await web3.eth.getBalance(address);

57+ let balance = toBN(await web3.eth.getBalance(address));

58+ cost = toBN(cost);

58− if (cost > balance) {

60+ if (cost.gt(balance)) {

59 try {

61 try {

60 if (point !== null) {

62 if (point !== null) {

61 const fundsRemaining = await remainingTransactions(point);

63 const fundsRemaining = await remainingTransactions(point);

@@ -77,36 +79,42 @@ const ensureFundsFor = async (

77 let newBalance = await web3.eth.getBalance(address);

79 let newBalance = await web3.eth.getBalance(address);

78 console.log(

80 console.log(

79 'tank: funds have confirmed',

81 'tank: funds have confirmed',

80− balance >= cost,

82+ balance.toString(),

81− balance,

82 newBalance

83 newBalance

83 );

84 );

84 return true;

85 return true;

85 }

86 }

86 } catch (e) {

87 } catch (e) {

87 console.log('tank: funding failed', e);

88 console.log('tank: funding failed', e);

88− await waitForBalance(web3, address, cost, askForFunding, gotFunding);

89+ await waitForBalance(

90+ web3,

91+ address,

92+ cost.toString(),

93+ askForFunding,

94+ gotFunding

95+ );

89 }

96 }

90 } else {

97 } else {

98+ console.log(

91− console.log('tank: already have sufficient funds', cost, address, balance);

99+ 'tank: already have sufficient funds',

100+ cost.toString(),

101+ address,

102+ balance.toString()

103+ );

92 }

104 }

93 return false;

105 return false;

94};

106};

107

96// returns a promise that resolves when address has at least minBalance

108// returns a promise that resolves when address has at least minBalance

97function waitForBalance(web3, address, minBalance, askForFunding, gotFunding) {

109function waitForBalance(web3, address, minBalance, askForFunding, gotFunding) {

110+ minBalance = toBN(minBalance);

98 return retry(async (bail, n) => {

111 return retry(async (bail, n) => {

99− const balance = await web3.eth.getBalance(address);

112+ const balance = toBN(await web3.eth.getBalance(address));

100− if (balance >= minBalance) {

113+ if (balance.gte(minBalance)) {

101 gotFunding && gotFunding();

114 gotFunding && gotFunding();

102 return;

115 return;

103 } else {

116 } else {

104− // TODO: minBalance is a `number` type

105− // but we want to display ETH, which will never accept a number

106− // but instead wants string or BN/BigNumber.

107− // this will be heavily refactored to correctly do BN math, but until

108− // then we'll manually convert this number to an integer string

109− askForFunding(address, minBalance.toFixed(), balance);

117+ askForFunding(address, minBalance, balance);

110 throw new Error('retrying');

118 throw new Error('retrying');

111 }

119 }

112 }, RETRY_OPTIONS);

120 }, RETRY_OPTIONS);

src/lib/useEthereumTransaction.js+3−2

@@ -1,6 +1,6 @@

1import { useCallback, useEffect, useState } from 'react';

2import { Just } from 'folktale/maybe';

3−import { toWei } from 'web3-utils';

3+import { toWei, toBN } from 'web3-utils';

5import { useNetwork } from 'store/network';

6import { useWallet } from 'store/wallet';

@@ -136,7 +136,8 @@ export default function useEthereumTransaction(

136 setError(undefined);

137

138 const rawTx = hexify(signedTransaction.serialize());

139− const cost = toWei(gasPrice.toFixed(0), 'gwei') * gasLimit;

139+ const costGwei = toBN(gasLimit).mul(toBN(gasPrice));

140+ const cost = toWei(costGwei.toString(), 'gwei');

140

141

141 let usedTank = false;

142 let usedTank = false;

142 // if this ethereum transaction is being executed by a specific point

143 // if this ethereum transaction is being executed by a specific point

src/views/Invite/InviteEmail.js+5−2

@@ -18,7 +18,7 @@ import {

18 AccessoryIcon,

19} from 'indigo-react';

20import { uniq } from 'lodash';

21−import { fromWei, toWei } from 'web3-utils';

21+import { fromWei, toWei, toBN } from 'web3-utils';

22import { FieldArray } from 'react-final-form-arrays';

24import { usePointCursor } from 'store/pointCursor';

@@ -306,11 +306,14 @@ export default function InviteEmail() {

306 }

307

308 setStatus(STATUS.FUNDING);

309+ const totalCost = toBN(GAS_LIMIT)

310+ .mul(toBN(gasPrice))

311+ .mul(toBN(emails.length));

309 const tankWasUsed = await tank.ensureFundsFor(

312 const tankWasUsed = await tank.ensureFundsFor(

310 _web3,

313 _web3,

311 point,

314 point,

312 _wallet.address,

315 _wallet.address,

313− toWei((gasPrice * GAS_LIMIT * emails.length).toFixed(), 'gwei'),

316+ toWei(totalCost.toString(), 'gwei'),

314 names.map(name => invites[name].rawTx),

317 names.map(name => invites[name].rawTx),

315 (address, minBalance, balance) =>

318 (address, minBalance, balance) =>

316 setNeedFunds({ address, minBalance, balance }),

319 setNeedFunds({ address, minBalance, balance }),