From 3e5b2a70e5592519ee6c6004d20ff9bba91c9b4d Mon Sep 17 00:00:00 2001
From: Callum McIntyre <callum.mcintyre@solana.com>
Date: Tue, 26 Sep 2023 18:31:51 +0100
Subject: [PATCH] Use UTL API directly for address token info (#298)

The problem here was not with our request coalescer, the
`getMultipleAccounts` requests are coming from the UTL SDK. Specifically
on the transaction page where we use `<Address pubkey={pubkey} ...
fetchTokenLabelInfo`. We do this on addresses that may be, but probably
are not, token mints, ie every address in a transaction.

Previously this check came from the baked in legacy token list. When the
token list was removed it was moved to the UTL SDK. The UTL SDK first
calls the UTL API to see if the address has known token info. If it does
not then it makes an on-chain call using `getMultipleAccounts` to fetch
the token info from the Metaplex metadata program if it exists.

I don't think this fallback is appropriate for our use case of
`fetchTokenLabelInfo` on addresses. Most addresses displayed are not
tokens, and we display many addresses simultaneously. This is leading to
many unnecessary `getMultipleAccounts` calls, and rate limiting.

Instead for this use case, we should only use the UTL API. This may mean
that brand new tokens can be displayed in eg the token details page, but
won't be correctly labelled in addresses. But this is a better tradeoff
than making an on-chain request for every address that we render that
may be a token.

Note that we already use the same public API for search
(https://github.com/solana-labs/explorer/blob/master/app/utils/token-search.ts),
where the SDK also can't provide an on-chain fallback.
---
 app/components/common/Address.tsx |  4 ++--
 app/utils/token-info.ts           | 36 ++++++++++++++++++++++++++++---
 2 files changed, 35 insertions(+), 5 deletions(-)

diff --git a/app/components/common/Address.tsx b/app/components/common/Address.tsx
index f0e84ffa..21c35deb 100644
--- a/app/components/common/Address.tsx
+++ b/app/components/common/Address.tsx
@@ -10,7 +10,7 @@ import React from 'react';
 import { useState } from 'react';
 import useAsyncEffect from 'use-async-effect';
 
-import { getTokenInfo } from '@/app/utils/token-info';
+import { getTokenInfoWithoutOnChainFallback } from '@/app/utils/token-info';
 
 import { Copyable } from './Copyable';
 
@@ -125,7 +125,7 @@ const useTokenInfo = (fetchTokenLabelInfo: boolean | undefined, pubkey: string)
         if (!fetchTokenLabelInfo) return;
         if (!info) {
             try {
-                const token = await getTokenInfo(new PublicKey(pubkey), cluster, url);
+                const token = await getTokenInfoWithoutOnChainFallback(new PublicKey(pubkey), cluster);
                 if (isMounted()) {
                     setInfo(token);
                 }
diff --git a/app/utils/token-info.ts b/app/utils/token-info.ts
index c1543fb6..ad3c2afb 100644
--- a/app/utils/token-info.ts
+++ b/app/utils/token-info.ts
@@ -73,6 +73,36 @@ export async function getTokenInfo(
     return token;
 }
 
+type UtlApiResponse = {
+    content: Token[]
+}
+
+export async function getTokenInfoWithoutOnChainFallback(
+    address: PublicKey,
+    cluster: Cluster
+): Promise<Token | undefined> {
+    const chainId = getChainId(cluster);
+    if (!chainId) return undefined;
+
+    // Request token info directly from UTL API
+    // We don't use the SDK here because we don't want it to fallback to an on-chain request
+    const response = await fetch(`https://token-list-api.solana.cloud/v1/mints?chainId=${chainId}`, {
+        body: JSON.stringify({ addresses: [address.toBase58()] }),
+        headers: {
+            "Content-Type": "application/json",
+        },
+        method: 'POST',
+    })
+
+    if (response.status >= 400) {
+        console.error(`Error calling UTL API for address ${address} on chain ID ${chainId}. Status ${response.status}`);
+        return undefined
+    }
+
+    const fetchedData = await response.json() as UtlApiResponse;
+    return fetchedData.content[0];
+}
+
 async function getFullLegacyTokenInfoUsingCdn(
     address: PublicKey,
     chainId: ChainId
@@ -111,9 +141,9 @@ export async function getFullTokenInfo(
     if (!sdkTokenInfo) {
         return legacyCdnTokenInfo
             ? {
-                  ...legacyCdnTokenInfo,
-                  verified: true,
-              }
+                ...legacyCdnTokenInfo,
+                verified: true,
+            }
             : undefined;
     }