Don't throw when RFQ-T client isn't whitelisted

Addresses review comments #2541 (comment) and #2541 (comment)
0xProject · Apr 9, 2020 · 5f4778c · 5f4778c
1 parent 70add44
commit 5f4778c
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/packages/asset-swapper/src/swap_quoter.ts b/packages/asset-swapper/src/swap_quoter.ts
@@ -532,10 +532,12 @@ export class SwapQuoter {
         const orderPromises: Array<Promise<SignedOrder[]>> = [
             this._getSignedOrdersAsync(makerAssetData, takerAssetData),
         ];
-        if (options.rfqt && options.rfqt.intentOnFilling && options.apiKey) {
-            if (!this._rfqtTakerApiKeyWhitelist.includes(options.apiKey)) {
-                throw new Error('API key not permissioned for RFQ-T');
-            }
+        if (
+            options.rfqt &&
+            options.rfqt.intentOnFilling &&
+            options.apiKey &&
+            this._rfqtTakerApiKeyWhitelist.includes(options.apiKey)
+        ) {
             if (!options.rfqt.takerAddress || options.rfqt.takerAddress === constants.NULL_ADDRESS) {
                 throw new Error('RFQ-T requests must specify a taker address');
             }