status-im / status-mobile

a free (libre) open source, mobile OS for Ethereum
https://status.app
Mozilla Public License 2.0
3.88k stars 984 forks source link

Exchange ERC-20 using wallet connect throws an error ("Something went wrong") #21137

Open churik opened 2 weeks ago

churik commented 2 weeks ago

Bug Report

Problem

At attempt to exchange any asset (not ETH) through decentralized exchanged (i.e. Uniswap or 0X) there is error at attempt to perform last step required for exchange

Expected behavior

can exchange Rare

Actual behavior

https://github.com/user-attachments/assets/9a6ddea4-4434-4bee-b1cd-648de96ac003

Reproduction

  1. Connect wallet to Uniswap
  2. Try to exchange any ERC20 to ETH

Additional Information

IMPORTANT NOTE: seems after a while it was possible to finish the transaction, but behavior makes user think that there is something wrong with a flow.

churik commented 2 weeks ago

cc @clauxx

clauxx commented 2 weeks ago

Most likely it's the issue of us using the old geth version, where not all typed data types were accepted as valid. If that's the case, the potential fix was already merged in status-go in this PR. Should test with the latest status-go to make sure it all works.