WalletConnect / blockchain-api

WalletConnect's Blockchain API
MIT License
51 stars 40 forks source link

fix(one_inch): using non-checksumed address only #692

Closed geekbrother closed 2 months ago

geekbrother commented 2 months ago

Description

This PR changing to use non-checksummed addresses only in requests to 1Inch endpoints. Some of the endpoints like the token price don't support checksummed addresses and return token not found error response when passing the checksummed token contract address. To avoid such collisions we should convert all checksummed addresses to non-checksummed by converting them to lowercase when requesting 1Inch.

How Has This Been Tested?

Due Diligence