From 32923ec7e14913db631d77252ca1b91eb79dc5a7 Mon Sep 17 00:00:00 2001 From: "F. Eugene Aumson" Date: Fri, 2 Oct 2020 14:18:31 -0400 Subject: [PATCH] Blacklist for timeout === max, not just > --- packages/asset-swapper/src/utils/quote_requestor.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/asset-swapper/src/utils/quote_requestor.ts b/packages/asset-swapper/src/utils/quote_requestor.ts index 529a890b59..ea889e5ebd 100644 --- a/packages/asset-swapper/src/utils/quote_requestor.ts +++ b/packages/asset-swapper/src/utils/quote_requestor.ts @@ -393,7 +393,7 @@ export class QuoteRequestor { }, }, }); - rfqMakerBlacklist.logTimeoutOrLackThereof(url, latencyMs > maxResponseTimeMs); + rfqMakerBlacklist.logTimeoutOrLackThereof(url, latencyMs >= maxResponseTimeMs); result.push({ response: response.data, makerUri: url }); } catch (err) { const latencyMs = Date.now() - timeBeforeAwait; @@ -409,7 +409,7 @@ export class QuoteRequestor { }, }, }); - rfqMakerBlacklist.logTimeoutOrLackThereof(url, latencyMs > maxResponseTimeMs); + rfqMakerBlacklist.logTimeoutOrLackThereof(url, latencyMs >= maxResponseTimeMs); this._warningLogger( convertIfAxiosError(err), `Failed to get RFQ-T ${quoteType} quote from market maker endpoint ${url} for API key ${