chainlist icon indicating copy to clipboard operation
chainlist copied to clipboard

Fixed Issue #1674

Open mubarakone opened this issue 5 months ago • 10 comments

https://github.com/DefiLlama/chainlist/issues/1674

issue1674

mubarakone avatar Jul 19 '25 02:07 mubarakone

transaction hash should be different for each chain right ? i see tx not found error Screenshot

mintdart avatar Jul 21 '25 10:07 mintdart

Yeah looks like each chain will need its own transaction hash, which I am still deciding how to go about it. Hardcoding every hash doesn't seem feasible

mubarakone avatar Jul 22 '25 01:07 mubarakone

Okay... turns out that transaction not found means that debug_traceTransaction is detected in the RPC, and is looking for the transaction based on the params. All we need is to indicate the debug_traceTransaction method exists, so the params is a dummy address. If we get in the error msg something like "method does not exist" or "API has no access", then we know for sure debug_traceTransaction does not exist and therefore no trace support for that RPC.

mubarakone avatar Aug 25 '25 02:08 mubarakone

The burn address is used to test for archive support, since it exists in every EVM chain.

mubarakone avatar Aug 25 '25 02:08 mubarakone

debug_transaction expects two arguments Screenshot

mintdart avatar Aug 25 '25 11:08 mintdart

thats okay because if debug_traceTransaction is recognized, that means trace support exist. I left the second argument blank because even if I did add a dummy variable, an error will still occur because the transaction never existed. But we aren't trying to look for a specific transaction, we are just trying to find if debug_traceTransaction method exists (which means trace support exists)

mubarakone avatar Aug 25 '25 18:08 mubarakone

there are archive nodes that do not support traces

mintdart avatar Aug 25 '25 19:08 mintdart

yes those are requested separately

mubarakone avatar Aug 25 '25 23:08 mubarakone

ACK on idea, if @Caranell's review is positive lets merge it

0xngmi avatar Sep 16 '25 19:09 0xngmi

Made some fixes:

  1. using same queryKey for both trace & archive queries caused the latter not to trigger at all & only the trace data being used for both metrics
  2. added more checks to trace debugging to filter out false positives/negatives (a bit debatable, feel free to comment)

Caranell avatar Sep 16 '25 21:09 Caranell