Commit 0a8a82e6 authored by tom goriunov's avatar tom goriunov Committed by GitHub

hotfix: fix error in degraded views when no RPC URL provided (#1583)

* typo fix

* fix block views

* fix tx views

---------
Co-authored-by: default avatarisstuev <natix.naf@gmail.com>
parent 504de39d
...@@ -2,10 +2,18 @@ import { createPublicClient, http } from 'viem'; ...@@ -2,10 +2,18 @@ import { createPublicClient, http } from 'viem';
import currentChain from './currentChain'; import currentChain from './currentChain';
export const publicClient = createPublicClient({ export const publicClient = (() => {
if (currentChain.rpcUrls.public.http.filter(Boolean).length === 0) {
return;
}
try {
return createPublicClient({
chain: currentChain, chain: currentChain,
transport: http(), transport: http(),
batch: { batch: {
multicall: true, multicall: true,
}, },
}); });
} catch (error) {}
})();
...@@ -50,6 +50,10 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery { ...@@ -50,6 +50,10 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery {
const rpcQuery = useQuery<RpcResponseType, unknown, Block | null>({ const rpcQuery = useQuery<RpcResponseType, unknown, Block | null>({
queryKey: [ 'RPC', 'block', { heightOrHash } ], queryKey: [ 'RPC', 'block', { heightOrHash } ],
queryFn: async() => { queryFn: async() => {
if (!publicClient) {
return null;
}
const blockParams = heightOrHash.startsWith('0x') ? { blockHash: heightOrHash as `0x${ string }` } : { blockNumber: BigInt(heightOrHash) }; const blockParams = heightOrHash.startsWith('0x') ? { blockHash: heightOrHash as `0x${ string }` } : { blockNumber: BigInt(heightOrHash) };
return publicClient.getBlock(blockParams).catch(() => null); return publicClient.getBlock(blockParams).catch(() => null);
}, },
...@@ -86,13 +90,13 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery { ...@@ -86,13 +90,13 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery {
}; };
}, },
placeholderData: GET_BLOCK, placeholderData: GET_BLOCK,
enabled: apiQuery.isError || apiQuery.errorUpdateCount > 0, enabled: publicClient !== undefined && (apiQuery.isError || apiQuery.errorUpdateCount > 0),
retry: false, retry: false,
refetchOnMount: false, refetchOnMount: false,
}); });
React.useEffect(() => { React.useEffect(() => {
if (apiQuery.isPlaceholderData) { if (apiQuery.isPlaceholderData || !publicClient) {
return; return;
} }
...@@ -109,7 +113,7 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery { ...@@ -109,7 +113,7 @@ export default function useBlockQuery({ heightOrHash }: Params): BlockQuery {
} }
}, [ rpcQuery.data, rpcQuery.isPlaceholderData ]); }, [ rpcQuery.data, rpcQuery.isPlaceholderData ]);
const isRpcQuery = Boolean((apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0 && rpcQuery.data); const isRpcQuery = Boolean(publicClient && (apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0 && rpcQuery.data);
const query = isRpcQuery ? rpcQuery as UseQueryResult<Block, ResourceError<{ status: number }>> : apiQuery; const query = isRpcQuery ? rpcQuery as UseQueryResult<Block, ResourceError<{ status: number }>> : apiQuery;
return { return {
......
...@@ -63,6 +63,10 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param ...@@ -63,6 +63,10 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param
const rpcQuery = useQuery<RpcResponseType, unknown, BlockTransactionsResponse | null>({ const rpcQuery = useQuery<RpcResponseType, unknown, BlockTransactionsResponse | null>({
queryKey: [ 'RPC', 'block_txs', { heightOrHash } ], queryKey: [ 'RPC', 'block_txs', { heightOrHash } ],
queryFn: async() => { queryFn: async() => {
if (!publicClient) {
return null;
}
const blockParams = heightOrHash.startsWith('0x') ? const blockParams = heightOrHash.startsWith('0x') ?
{ blockHash: heightOrHash as `0x${ string }`, includeTransactions: true } : { blockHash: heightOrHash as `0x${ string }`, includeTransactions: true } :
{ blockNumber: BigInt(heightOrHash), includeTransactions: true }; { blockNumber: BigInt(heightOrHash), includeTransactions: true };
...@@ -125,13 +129,13 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param ...@@ -125,13 +129,13 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param
}; };
}, },
placeholderData: GET_BLOCK_WITH_TRANSACTIONS, placeholderData: GET_BLOCK_WITH_TRANSACTIONS,
enabled: tab === 'txs' && (blockQuery.isDegradedData || apiQuery.isError || apiQuery.errorUpdateCount > 0), enabled: publicClient !== undefined && tab === 'txs' && (blockQuery.isDegradedData || apiQuery.isError || apiQuery.errorUpdateCount > 0),
retry: false, retry: false,
refetchOnMount: false, refetchOnMount: false,
}); });
React.useEffect(() => { React.useEffect(() => {
if (apiQuery.isPlaceholderData) { if (apiQuery.isPlaceholderData || !publicClient) {
return; return;
} }
...@@ -151,7 +155,7 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param ...@@ -151,7 +155,7 @@ export default function useBlockTxQuery({ heightOrHash, blockQuery, tab }: Param
const isRpcQuery = Boolean(( const isRpcQuery = Boolean((
blockQuery.isDegradedData || blockQuery.isDegradedData ||
((apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0) ((apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0)
) && rpcQuery.data); ) && rpcQuery.data && publicClient);
const rpcQueryWithPages: QueryWithPagesResult<'block_txs'> = React.useMemo(() => { const rpcQueryWithPages: QueryWithPagesResult<'block_txs'> = React.useMemo(() => {
return { return {
......
...@@ -65,6 +65,10 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab ...@@ -65,6 +65,10 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab
const rpcQuery = useQuery<RpcResponseType, unknown, BlockWithdrawalsResponse | null>({ const rpcQuery = useQuery<RpcResponseType, unknown, BlockWithdrawalsResponse | null>({
queryKey: [ 'RPC', 'block', { heightOrHash } ], queryKey: [ 'RPC', 'block', { heightOrHash } ],
queryFn: async() => { queryFn: async() => {
if (!publicClient) {
return null;
}
const blockParams = heightOrHash.startsWith('0x') ? { blockHash: heightOrHash as `0x${ string }` } : { blockNumber: BigInt(heightOrHash) }; const blockParams = heightOrHash.startsWith('0x') ? { blockHash: heightOrHash as `0x${ string }` } : { blockNumber: BigInt(heightOrHash) };
return publicClient.getBlock(blockParams).catch(() => null); return publicClient.getBlock(blockParams).catch(() => null);
}, },
...@@ -89,6 +93,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab ...@@ -89,6 +93,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab
}, },
placeholderData: GET_BLOCK, placeholderData: GET_BLOCK,
enabled: enabled:
publicClient !== undefined &&
tab === 'withdrawals' && tab === 'withdrawals' &&
config.features.beaconChain.isEnabled && config.features.beaconChain.isEnabled &&
(blockQuery.isDegradedData || apiQuery.isError || apiQuery.errorUpdateCount > 0), (blockQuery.isDegradedData || apiQuery.isError || apiQuery.errorUpdateCount > 0),
...@@ -97,7 +102,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab ...@@ -97,7 +102,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab
}); });
React.useEffect(() => { React.useEffect(() => {
if (apiQuery.isPlaceholderData) { if (apiQuery.isPlaceholderData || !publicClient) {
return; return;
} }
...@@ -117,7 +122,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab ...@@ -117,7 +122,7 @@ export default function useBlockWithdrawalsQuery({ heightOrHash, blockQuery, tab
const isRpcQuery = Boolean(( const isRpcQuery = Boolean((
blockQuery.isDegradedData || blockQuery.isDegradedData ||
((apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0) ((apiQuery.isError || apiQuery.isPlaceholderData) && apiQuery.errorUpdateCount > 0)
) && rpcQuery.data); ) && rpcQuery.data && publicClient);
const rpcQueryWithPages: QueryWithPagesResult<'block_withdrawals'> = React.useMemo(() => { const rpcQueryWithPages: QueryWithPagesResult<'block_withdrawals'> = React.useMemo(() => {
return { return {
......
...@@ -7,6 +7,7 @@ import config from 'configs/app'; ...@@ -7,6 +7,7 @@ import config from 'configs/app';
import { useAppContext } from 'lib/contexts/app'; import { useAppContext } from 'lib/contexts/app';
import throwOnResourceLoadError from 'lib/errors/throwOnResourceLoadError'; import throwOnResourceLoadError from 'lib/errors/throwOnResourceLoadError';
import getQueryParamString from 'lib/router/getQueryParamString'; import getQueryParamString from 'lib/router/getQueryParamString';
import { publicClient } from 'lib/web3/client';
import TextAd from 'ui/shared/ad/TextAd'; import TextAd from 'ui/shared/ad/TextAd';
import EntityTags from 'ui/shared/EntityTags'; import EntityTags from 'ui/shared/EntityTags';
import PageTitle from 'ui/shared/Page/PageTitle'; import PageTitle from 'ui/shared/Page/PageTitle';
...@@ -33,7 +34,7 @@ const TransactionPageContent = () => { ...@@ -33,7 +34,7 @@ const TransactionPageContent = () => {
const txQuery = useTxQuery(); const txQuery = useTxQuery();
const { data, isPlaceholderData, isError, error, errorUpdateCount } = txQuery; const { data, isPlaceholderData, isError, error, errorUpdateCount } = txQuery;
const showDegradedView = (isError || isPlaceholderData) && errorUpdateCount > 0; const showDegradedView = publicClient && (isError || isPlaceholderData) && errorUpdateCount > 0;
const tabs: Array<RoutedTab> = (() => { const tabs: Array<RoutedTab> = (() => {
const detailsComponent = showDegradedView ? const detailsComponent = showDegradedView ?
...@@ -97,9 +98,11 @@ const TransactionPageContent = () => { ...@@ -97,9 +98,11 @@ const TransactionPageContent = () => {
return <RoutedTabs tabs={ tabs }/>; return <RoutedTabs tabs={ tabs }/>;
})(); })();
if (error?.status === 422) { if (isError && !showDegradedView) {
if (error?.status === 422 || error?.status === 404) {
throwOnResourceLoadError({ resource: 'tx', error, isError: true }); throwOnResourceLoadError({ resource: 'tx', error, isError: true });
} }
}
return ( return (
<> <>
......
import React from 'react'; import React from 'react';
import TestnetWarning from 'ui/shared/alerts/TestnetWarning'; import TestnetWarning from 'ui/shared/alerts/TestnetWarning';
import DataFetchAlert from 'ui/shared/DataFetchAlert';
import TxInfo from './details/TxInfo'; import TxInfo from './details/TxInfo';
import type { TxQuery } from './useTxQuery'; import type { TxQuery } from './useTxQuery';
...@@ -10,6 +11,10 @@ interface Props { ...@@ -10,6 +11,10 @@ interface Props {
} }
const TxDetails = ({ txQuery }: Props) => { const TxDetails = ({ txQuery }: Props) => {
if (txQuery.isError) {
return <DataFetchAlert/>;
}
return ( return (
<> <>
<TestnetWarning mb={ 6 } isLoading={ txQuery.isPlaceholderData }/> <TestnetWarning mb={ 6 } isLoading={ txQuery.isPlaceholderData }/>
......
...@@ -37,6 +37,10 @@ const TxDetailsDegraded = ({ hash, txQuery }: Props) => { ...@@ -37,6 +37,10 @@ const TxDetailsDegraded = ({ hash, txQuery }: Props) => {
const query = useQuery<RpcResponseType, unknown, Transaction | null>({ const query = useQuery<RpcResponseType, unknown, Transaction | null>({
queryKey: [ 'RPC', 'tx', { hash } ], queryKey: [ 'RPC', 'tx', { hash } ],
queryFn: async() => { queryFn: async() => {
if (!publicClient) {
throw new Error('No public RPC client');
}
const tx = await publicClient.getTransaction({ hash: hash as `0x${ string }` }); const tx = await publicClient.getTransaction({ hash: hash as `0x${ string }` });
if (!tx) { if (!tx) {
......
...@@ -2,6 +2,7 @@ import React from 'react'; ...@@ -2,6 +2,7 @@ import React from 'react';
import { USER_OPS_ITEM } from 'stubs/userOps'; import { USER_OPS_ITEM } from 'stubs/userOps';
import { generateListStub } from 'stubs/utils'; import { generateListStub } from 'stubs/utils';
import DataFetchAlert from 'ui/shared/DataFetchAlert';
import useQueryWithPages from 'ui/shared/pagination/useQueryWithPages'; import useQueryWithPages from 'ui/shared/pagination/useQueryWithPages';
import TxPendingAlert from 'ui/tx/TxPendingAlert'; import TxPendingAlert from 'ui/tx/TxPendingAlert';
import TxSocketAlert from 'ui/tx/TxSocketAlert'; import TxSocketAlert from 'ui/tx/TxSocketAlert';
...@@ -28,6 +29,10 @@ const TxUserOps = ({ txQuery }: Props) => { ...@@ -28,6 +29,10 @@ const TxUserOps = ({ txQuery }: Props) => {
return txQuery.socketStatus ? <TxSocketAlert status={ txQuery.socketStatus }/> : <TxPendingAlert/>; return txQuery.socketStatus ? <TxSocketAlert status={ txQuery.socketStatus }/> : <TxPendingAlert/>;
} }
if (txQuery.isError) {
return <DataFetchAlert/>;
}
return <UserOpsContent query={ userOpsQuery } showTx={ false }/>; return <UserOpsContent query={ userOpsQuery } showTx={ false }/>;
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment