Publish

- @0x/asset-swapper@16.49.2
Updated CHANGELOGS & MD docs
2022-01-31 18:24:47 +00:00 · 2022-01-31 18:24:44 +00:00 · 2022-01-31 12:51:17 -05:00
4 changed files with 25 additions and 3 deletions
--- a/packages/asset-swapper/CHANGELOG.json
+++ b/packages/asset-swapper/CHANGELOG.json
@@ -1,4 +1,14 @@
 [
+    {
+        "version": "16.49.2",
+        "changes": [
+            {
+                "note": "Fix ABI encoding error with two hop buys due to applying slippage to uint(-1) values",
+                "pr": 410
+            }
+        ],
+        "timestamp": 1643653482
+    },
    {
        "version": "16.49.1",
        "changes": [
--- a/packages/asset-swapper/CHANGELOG.md
+++ b/packages/asset-swapper/CHANGELOG.md
@@ -5,6 +5,10 @@ Edit the package's CHANGELOG.json file only.

 CHANGELOG

+## v16.49.2 - _January 31, 2022_
+
+    * Fix ABI encoding error with two hop buys due to applying slippage to uint(-1) values (#410)
+
 ## v16.49.1 - _January 31, 2022_

    * Fix WorstCaseQuoteInfo encoding bug (#402)
--- a/packages/asset-swapper/package.json
+++ b/packages/asset-swapper/package.json
@@ -1,6 +1,6 @@
 {
    "name": "@0x/asset-swapper",
-    "version": "16.49.1",
+    "version": "16.49.2",
    "engines": {
        "node": ">=6.12"
    },
--- a/packages/asset-swapper/src/quote_consumers/exchange_proxy_swap_quote_consumer.ts
+++ b/packages/asset-swapper/src/quote_consumers/exchange_proxy_swap_quote_consumer.ts
@@ -701,8 +701,16 @@ function slipNonNativeOrders(quote: MarketSellSwapQuote | MarketBuySwapQuote): O
        return {
            ...o,
            ...(quote.type === MarketOperation.Sell
-                ? { makerAmount: o.makerAmount.times(1 - slippage).integerValue(BigNumber.ROUND_DOWN) }
-                : { takerAmount: o.takerAmount.times(1 + slippage).integerValue(BigNumber.ROUND_UP) }),
+                ? {
+                      makerAmount: o.makerAmount.eq(MAX_UINT256)
+                          ? MAX_UINT256
+                          : o.makerAmount.times(1 - slippage).integerValue(BigNumber.ROUND_DOWN),
+                  }
+                : {
+                      takerAmount: o.takerAmount.eq(MAX_UINT256)
+                          ? MAX_UINT256
+                          : o.takerAmount.times(1 + slippage).integerValue(BigNumber.ROUND_UP),
+                  }),
        };
    });
 }
Author	SHA1	Message	Date
Github Actions	0f701f42d3	Publish - @0x/asset-swapper@16.49.2	2022-01-31 18:24:47 +00:00
Github Actions	3e3e82d3f7	Updated CHANGELOGS & MD docs	2022-01-31 18:24:44 +00:00
Lawrence Forman	c57bf86273	Fix ABI encoding error with two hop buys due to applying slippage to uint(-1) values (#410 ) Co-authored-by: Lawrence Forman <me@merklejerk.com>	2022-01-31 12:51:17 -05:00