Consensys / Consensys/abi-decoder

Multiple signature of the same events (with and without index) not processed correctly

Open
#43 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
641
Forks
215
PR merge metrics
No merged PRs in 30d

Description

In case an event (log) has a signature of eventName(address arg1) and eventName(indexed address arg1) - the normalised signature is the same (lets say X)

loading multiple ABIs does not help as the latest ABI is saved for handling the X form

Need to save an array of ABIs for each method/log
try handle them in series until one succeed or all of them failed

https://etherscan.io/tx/0xe9dc240d688fb3aa78f2d18bc73812f613f564dcd1768e4e4b96e5ea85c7c6e1

This is an example of a transaction that fails to process
Log 2 and Log 10 are the same, but one has indexed args and the other not
the library tries to handle both of them as indexed so it throws on this line(168)

decodedP.value = decodedP.value.toLowerCase();

since value is undefined

** Relevant probably only for address type

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.