Remove exchange rate retry

With the cache, and periodic looping, this isn't really necessary and can actually slow down rate updates

Author: peachbits

src/actions/ExchangeRateActions.ts

@@ -310,103 +310,101 @@ async function fetchExchangeRates(
 
   const requests = convertToRatesParams(cryptoPairMap, fiatPairMap)
   for (const query of requests) {
-    for (let attempt = 0; attempt < 5; ++attempt) {
-      const options = {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify(query)
-      }
-      try {
-        const response = await fetchRates('v3/rates', options)
-        if (response.ok) {
-          const json = await response.json()
-          const cleanedRates = asRatesParams(json)
-          const targetFiat = fixFiatCurrencyCode(cleanedRates.targetFiat)
-
-          for (const cryptoRate of cleanedRates.crypto) {
-            const { asset, isoDate, rate } = cryptoRate
-            if (rate == null) continue
-
-            const { pluginId, tokenId } = asset
-            const safeTokenId = tokenId ?? ''
-
-            rates.crypto[pluginId] ??= {}
-            rates.crypto[pluginId][safeTokenId] ??= {}
-            rates.crypto[pluginId][safeTokenId][targetFiat] ??= {
-              current: 0,
-              yesterday: 0,
-              yesterdayTimestamp: 0,
-              expiration: 0
-            }
+    const options = {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify(query)
+    }
+    try {
+      const response = await fetchRates('v3/rates', options)
+      if (response.ok) {
+        const json = await response.json()
+        const cleanedRates = asRatesParams(json)
+        const targetFiat = fixFiatCurrencyCode(cleanedRates.targetFiat)
+
+        for (const cryptoRate of cleanedRates.crypto) {
+          const { asset, isoDate, rate } = cryptoRate
+          if (rate == null) continue
+
+          const { pluginId, tokenId } = asset
+          const safeTokenId = tokenId ?? ''
+
+          rates.crypto[pluginId] ??= {}
+          rates.crypto[pluginId][safeTokenId] ??= {}
+          rates.crypto[pluginId][safeTokenId][targetFiat] ??= {
+            current: 0,
+            yesterday: 0,
+            yesterdayTimestamp: 0,
+            expiration: 0
+          }
 
-            const rateObj = rates.crypto[pluginId][safeTokenId][targetFiat]
-
-            const isHistorical =
-              isoDate != null && isoDate.getTime() < now - ONE_HOUR
-            if (isHistorical) {
-              const dateTimestamp = isoDate.getTime()
-              const yesterdayTargetTimestamp = Date.parse(yesterday)
-              const yesterdayRateTimestamp = rateObj.yesterdayTimestamp
-
-              // update yesterday rate if we find one closer than we have
-              if (
-                Math.abs(yesterdayTargetTimestamp - dateTimestamp) <
-                Math.abs(yesterdayTargetTimestamp - yesterdayRateTimestamp)
-              ) {
-                rates.crypto[pluginId][safeTokenId][
-                  targetFiat
-                ].yesterdayTimestamp = yesterdayTimestamp
-                rateObj.yesterday = rate
-              }
-            } else {
-              rateObj.current = rate
+          const rateObj = rates.crypto[pluginId][safeTokenId][targetFiat]
+
+          const isHistorical =
+            isoDate != null && isoDate.getTime() < now - ONE_HOUR
+          if (isHistorical) {
+            const dateTimestamp = isoDate.getTime()
+            const yesterdayTargetTimestamp = Date.parse(yesterday)
+            const yesterdayRateTimestamp = rateObj.yesterdayTimestamp
+
+            // update yesterday rate if we find one closer than we have
+            if (
+              Math.abs(yesterdayTargetTimestamp - dateTimestamp) <
+              Math.abs(yesterdayTargetTimestamp - yesterdayRateTimestamp)
+            ) {
+              rates.crypto[pluginId][safeTokenId][
+                targetFiat
+              ].yesterdayTimestamp = yesterdayTimestamp
+              rateObj.yesterday = rate
             }
+          } else {
+            rateObj.current = rate
+          }
 
-            rateObj.expiration = rateExpiration
+          rateObj.expiration = rateExpiration
+        }
+        for (const fiatRate of cleanedRates.fiat) {
+          const { isoDate, rate } = fiatRate
+          const fiatCode = fixFiatCurrencyCode(fiatRate.fiatCode)
+          if (rate == null) continue
+
+          rates.fiat[fiatCode] ??= {}
+          rates.fiat[fiatCode][targetFiat] ??= {
+            current: 0,
+            yesterday: 0,
+            yesterdayTimestamp: 0,
+            expiration: 0
           }
-          for (const fiatRate of cleanedRates.fiat) {
-            const { isoDate, rate } = fiatRate
-            const fiatCode = fixFiatCurrencyCode(fiatRate.fiatCode)
-            if (rate == null) continue
-
-            rates.fiat[fiatCode] ??= {}
-            rates.fiat[fiatCode][targetFiat] ??= {
-              current: 0,
-              yesterday: 0,
-              yesterdayTimestamp: 0,
-              expiration: 0
-            }
-            const rateObj = rates.fiat[fiatCode][targetFiat]
-
-            const isHistorical =
-              isoDate != null && isoDate.getTime() < now - ONE_HOUR
-            if (isHistorical) {
-              const dateTimestamp = isoDate.getTime()
-              const yesterdayTargetTimestamp = Date.parse(yesterday)
-              const yesterdayRateTimestamp = rateObj.yesterdayTimestamp
-
-              // update yesterday rate if we find one closer than we have
-              if (
-                Math.abs(yesterdayTargetTimestamp - dateTimestamp) <
-                Math.abs(yesterdayTargetTimestamp - yesterdayRateTimestamp)
-              ) {
-                rates.fiat[fiatCode][targetFiat].yesterdayTimestamp =
-                  yesterdayTimestamp
-                rateObj.yesterday = rate
-              }
-            } else {
-              rateObj.current = rate
+          const rateObj = rates.fiat[fiatCode][targetFiat]
+
+          const isHistorical =
+            isoDate != null && isoDate.getTime() < now - ONE_HOUR
+          if (isHistorical) {
+            const dateTimestamp = isoDate.getTime()
+            const yesterdayTargetTimestamp = Date.parse(yesterday)
+            const yesterdayRateTimestamp = rateObj.yesterdayTimestamp
+
+            // update yesterday rate if we find one closer than we have
+            if (
+              Math.abs(yesterdayTargetTimestamp - dateTimestamp) <
+              Math.abs(yesterdayTargetTimestamp - yesterdayRateTimestamp)
+            ) {
+              rates.fiat[fiatCode][targetFiat].yesterdayTimestamp =
+                yesterdayTimestamp
+              rateObj.yesterday = rate
             }
-
-            rateObj.expiration = rateExpiration
+          } else {
+            rateObj.current = rate
           }
-          break
+
+          rateObj.expiration = rateExpiration
         }
-      } catch (error: unknown) {
-        console.log(
-          `buildExchangeRates error querying rates server ${String(error)}`
-        )
+        break
       }
+    } catch (error: unknown) {
+      console.log(
+        `buildExchangeRates error querying rates server ${String(error)}`
+      )
     }
   }