search

safe-global / safe-android

🔐 App to manage and interact with Safe
https://safe.global
GNU General Public License v3.0
91 stars 38 forks source link

[Android][Tx execution]Goerli/Arbitrum- Estimation failed for the valid tx #1981

Open liliya-soroka opened 1 year ago

liliya-soroka commented 1 year ago

Describe the bug Goerli- Estimation failed for the valid tx

To Reproduce Precondition:

  1. owner key imported

  2. tx nonce =454

  3. send token Poly Steps to reproduce the behavior:

  4. Go to gor:0xC642bEc0e4f69815e89723C09caA8DAa524cAc9c on prod CGW

  5. Click Execute for the send 0.00001 Poly Current result: Estimation failed

  6. Click Estimated fee to see where the problem with estimation Current result: I can not open Edit estimation fee

Expected behavior The valid tx should be estimated without the problem

Smartphone (please complete the following information):

Environment prod app Additional context Add any other context about the problem here.

liliya-soroka commented 1 year ago

Same on Arbitrum : arb1:0x861099a9FD94b03471a1002efCa662c3ECAdB10C deposit wrapped ether ( tx nonce =67) - prod app

liliya-soroka commented 1 year ago

send back, as both txs are valid and it should be possible to execute them

  1. Goerli: the tx is executable. The gas limit estimation is wrong for this tx. Check the ios app implementation to adjust and fix it properly. Also , if the user increase gas limit manually to the correct amount it should be possible to execute tx( unblock the button and don't show the error)
  2. Same with Arbitrum. Please, check the behavior for the same tx in the ios app

app version 3.8.0 -4004

liliya-soroka commented 1 year ago

Verified app version 3.8.0 -4008