Consensys / Consensys/abi-decoder

_decodeLogs error on parsing hex topics

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

Description

abi-decoder has a dependency: "web3-utils": "^1.2.1".
That caused usage of web3-utils v1.7.3. Now it causes usage of web3-utils v1.7.4.
One of the changes between these versions is a wrapper around BN class added. The earlier version passed the args to the class through, but the new one tries to guess the base by looking "0x" at the beginning of the string.
Since abi-decoder removes "0x" from the value, bn.js tries to parse the value with base 10 and throws an error if the value contains letters or parses it with wrong base if doesn't.

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.