New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug] Ethereum Provider - Address is not in lower case #505

Open

Grinion opened this issue Feb 22, 2024 · 0 comments

Grinion commented Feb 22, 2024

Hello,

At runtime, this.address parameter contains a value that is not in the lowercase - therefore this condition cannot be true.

trust-web3-provider/src/ethereum_provider.js

Line 305 in 2bfd36a

this.address === payload.params?.[0].toLowerCase()

In the method setAddress that sets this value, it is converted to lowercase. There may be other places where this value is set.
https://github.com/trustwallet/trust-web3-provider/blob/2bfd36ade2ac2275782969dcae5ac6bf3d608054/src/ethereum_provider.js#L31C3-L31C13

Grinion mentioned this issue

Temporary fix trust wallet InjectiveLabs/injective-ts#400

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment