fix: try different logging approach

synapsecns · Dec 20, 2024 · 6f8da0b · 6f8da0b
1 parent 60393c7
commit 6f8da0b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 26 deletions.
diff --git a/packages/sdk-router/src/rfq/api.ts b/packages/sdk-router/src/rfq/api.ts
@@ -88,10 +88,7 @@ export const getAllQuotes = async (): Promise<FastBridgeQuote[]> => {
     }
     // The response is a list of quotes in the FastBridgeQuoteAPI format
     const quotes: FastBridgeQuoteAPI[] = await response.json()
-    console.log(
-      `${API_URL}/quotes with response:`,
-      JSON.stringify(quotes, null, 2)
-    )
+    console.log(`${API_URL}/quotes`, { quotes })
     return quotes
       .map((quote) => {
         try {
@@ -145,35 +142,28 @@ export const getBestRelayerQuote = async (
       },
     })
     if (!response.ok) {
-      console.error('Error fetching quote:', response.statusText)
+      console.error('Error fetching quote', { response })
       return ZeroQuote
     }
     // Check that response is successful, contains non-zero dest amount, and has a relayer address
     const rfqResponse: PutRFQResponseAPI = await response.json()
-    console.log(
-      `${API_URL}/rfq with request:`,
-      JSON.stringify(rfqRequest, null, 2),
-      '\n',
-      'response: ',
-      JSON.stringify(rfqResponse, null, 2)
-    )
+    console.log(`${API_URL}/rfq`, { rfqRequest, rfqResponse })
     if (!rfqResponse.success) {
-      console.error(
-        'No RFQ quote returned:',
-        rfqResponse.reason ?? 'Unknown reason'
-      )
+      console.error('No RFQ quote returned', {
+        reason: rfqResponse.reason ?? 'Unknown reason',
+      })
       return ZeroQuote
     }
     if (!rfqResponse.dest_amount || !rfqResponse.relayer_address) {
       console.error(
         'Error fetching quote: missing dest_amount or relayer_address in response:',
-        rfqResponse
+        { rfqResponse }
       )
       return ZeroQuote
     }
     const destAmount = BigNumber.from(rfqResponse.dest_amount)
     if (destAmount.lte(0)) {
-      console.error('No RFQ quote returned')
+      console.error('No RFQ quote returned', { rfqResponse })
       return ZeroQuote
     }
     return {
@@ -182,7 +172,7 @@ export const getBestRelayerQuote = async (
       quoteID: rfqResponse.quote_id,
     }
   } catch (error) {
-    console.error('Error fetching quote:', error)
+    console.error('Error fetching quote:', { error })
     return ZeroQuote
   }
 }
diff --git a/packages/sdk-router/src/rfq/engine/paraSwapEngine.ts b/packages/sdk-router/src/rfq/engine/paraSwapEngine.ts
@@ -151,27 +151,24 @@ export class ParaSwapEngine implements SwapEngine {
       if (request.slippage > MAX_SLIPPAGE) {
         request.slippage = MAX_SLIPPAGE
       }
+      console.log('Fetching ParaSwap response', { request })
       // Stringify every value in the request
       const params = new URLSearchParams(
         Object.entries(request).map(([k, v]) => {
           return [k, v.toString()]
         })
       )
       const url = `${PARASWAP_API_URL}?${params.toString()}`
-      console.log(`Fetching ParaSwap response from ${url}`)
       const response = await fetchWithTimeout(url, PARASWAP_API_TIMEOUT)
       if (!response.ok) {
-        console.error(`Error fetching ParaSwap response from ${url}:`, response)
+        console.error('Error fetching ParaSwap response', { url, response })
         return EmptyParaSwapResponse
       }
       const paraSwapResponse: ParaSwapResponse = await response.json()
-      console.log(
-        `Fetched ParaSwap response from ${url}:`,
-        JSON.stringify(paraSwapResponse, null, 2)
-      )
+      console.log('Fetched ParaSwap response', { url, paraSwapResponse })
       return paraSwapResponse
     } catch (error) {
-      console.error('Error fetching ParaSwap response:', error)
+      console.error('Error fetching ParaSwap response', { error })
       return EmptyParaSwapResponse
     }
   }