diff options
author | Brandon Millman <brandon.millman@gmail.com> | 2018-10-18 01:03:46 +0800 |
---|---|---|
committer | Brandon Millman <brandon.millman@gmail.com> | 2018-10-24 01:02:00 +0800 |
commit | 117ee19370e56f3559b9b11b04becc5ad9c7d634 (patch) | |
tree | 2f910b7dc26a155b367e088bc1dc02d9a58af623 /packages/asset-buyer/src/asset_buyer.ts | |
parent | dcd428a4a299de0daf6bbfb8e8a7c15685b673cf (diff) | |
download | dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar.gz dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar.bz2 dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar.lz dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar.xz dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.tar.zst dexon-sol-tools-117ee19370e56f3559b9b11b04becc5ad9c7d634.zip |
feat(asset-buyer): throw SignatureRequestDenied and TransactionValueTooLow errors when executing buy
Diffstat (limited to 'packages/asset-buyer/src/asset_buyer.ts')
-rw-r--r-- | packages/asset-buyer/src/asset_buyer.ts | 43 |
1 files changed, 27 insertions, 16 deletions
diff --git a/packages/asset-buyer/src/asset_buyer.ts b/packages/asset-buyer/src/asset_buyer.ts index 2fe8f6deb..2f9a3108e 100644 --- a/packages/asset-buyer/src/asset_buyer.ts +++ b/packages/asset-buyer/src/asset_buyer.ts @@ -1,4 +1,4 @@ -import { ContractWrappers } from '@0x/contract-wrappers'; +import { ContractWrappers, ContractWrappersError, ForwarderWrapperError } from '@0x/contract-wrappers'; import { schemas } from '@0x/json-schemas'; import { SignedOrder } from '@0x/order-utils'; import { ObjectMap } from '@0x/types'; @@ -210,21 +210,32 @@ export class AssetBuyer { throw new Error(AssetBuyerError.NoAddressAvailable); } } - // if no ethAmount is provided, default to the worst ethAmount from buyQuote - const txHash = await this._contractWrappers.forwarder.marketBuyOrdersWithEthAsync( - orders, - assetBuyAmount, - finalTakerAddress, - ethAmount || worstCaseQuoteInfo.totalEthAmount, - feeOrders, - feePercentage, - feeRecipient, - { - gasLimit, - gasPrice, - }, - ); - return txHash; + try { + // if no ethAmount is provided, default to the worst ethAmount from buyQuote + const txHash = await this._contractWrappers.forwarder.marketBuyOrdersWithEthAsync( + orders, + assetBuyAmount, + finalTakerAddress, + ethAmount || worstCaseQuoteInfo.totalEthAmount, + feeOrders, + feePercentage, + feeRecipient, + { + gasLimit, + gasPrice, + shouldValidate: true, + }, + ); + return txHash; + } catch (err) { + if (_.includes(err.message, ContractWrappersError.SignatureRequestDenied)) { + throw new Error(AssetBuyerError.SignatureRequestDenied); + } else if (_.includes(err.message, ForwarderWrapperError.CompleteFillFailed)) { + throw new Error(AssetBuyerError.TransactionValueTooLow); + } else { + throw err; + } + } } /** * Grab orders from the map, if there is a miss or it is time to refresh, fetch and process the orders |