crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

asset-swapper: fix bug: RFQ-T info log was broken

Browse Source
This commit is contained in:
F. Eugene Aumson
2020-05-11 17:39:30 -04:00
parent 9a7d9abe3c
commit b95cec9c32
1 changed files with 18 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
packages/asset-swapper/src/utils/quote_requestor.ts
View File

@@ -285,7 +285,10 @@ export class QuoteRequestor {
const responsesIfDefined: Array<undefined | AxiosResponse<ResponseT>> = await Promise.all(
Object.keys(this._rfqtAssetOfferings).map(async url => {
if (this._makerSupportsPair(url, makerAssetData, takerAssetData)) {
let logEntry;
const partialLogEntry = {
makerEndpoint: url,
quoteType,
};
const timeBeforeAwait = Date.now();
try {
const quotePath = (() => {
@@ -308,16 +311,22 @@ export class QuoteRequestor {
},
timeout: options.makerEndpointMaxResponseTimeMs,
});
logEntry = {
statusCode: response.status,
latency: Date.now() - timeBeforeAwait,
};
this._infoLogger({
rfqtMakerResponse: {
...partialLogEntry,
statusCode: response.status,
latency: Date.now() - timeBeforeAwait,
},
});
return response;
} catch (err) {
logEntry = {
statusCode: err.code,
latency: Date.now() - timeBeforeAwait,
};
this._infoLogger({
rfqtMakerResponse: {
...partialLogEntry,
statusCode: err.code,
latency: Date.now() - timeBeforeAwait,
},
});
this._warningLogger(
convertIfAxiosError(err),
`Failed to get RFQ-T ${quoteType} quote from market maker endpoint ${url} for API key ${
@@ -326,7 +335,6 @@ export class QuoteRequestor {
);
return undefined;
}
this._infoLogger({ rfqtMakerResponse: { ...logEntry, makerEndpoint: url, quoteType } });
}
return undefined;
}),