wormhole-foundation / wormhole-connect

Wormhole Connect brings all the functionality and utility of Wormhole right into your application and removes all of the complexity.
https://connect-in-style.wormhole.com/
Other
81 stars 65 forks source link

[QA][Mainnet] Transaction fails with reason - "Cannot estimate gas" #1710

Closed AigaD closed 6 months ago

AigaD commented 8 months ago

Preconditions: Metamask wallet connected at sending From section Sui Metamask Snap wallet connected at sending To section Console is open

Environment: MacBook Pro, macOS Ventura 13.0.1, Chrome Version 121.0.6167.85

Tested from: https://wormhole-connect-mainnet.netlify.app/# - Tested NOK https://preview.portalbridge.com/7246284e5d/# - Tested NOK https://wormhole-connect.netlify.app/ - Tested OK

Steps: Prepare to send 0.1 USDT from Ethereum (Metamask) to Sui (Sui Metamask Snap) Confirm that Manual Bridge in Route section is selected Click Approve and proceed with transaction Observe

Actual result: Transaction fails to start, failing reason: Error: cannot estimate gas; transaction may fail or may require manual gas limit

Expected result: Transaction is started

Comments: Transaction with automatic route tests OK.

Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/db83f2ed-c201-4131-9aa2-816612efc91b

cannot estimate gas

AigaD commented 8 months ago

Also transaction 0.1 USDT from Ethereum to Aptos (Nightly) using Manual Bridge in routes failed with reason: Error: cannot estimate gas; transaction may fail or may require manual gas limit Tested from: https://wormhole-connect-mainnet.netlify.app/# Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/e8ccb4bf-f1fa-4ec4-9660-2e1dcba6278c

tsadovska commented 8 months ago

Issue is reproducible on https://portalbridge.com/ [Mainnet] when sending 1 USDT from Ethereum (MetaMask) to Injective (Leap).

Error: cannot estimate gas; transaction may fail or may require manual gas limit [ See: https://links.ethers.org/v5-errors-UNPREDICTABLE_GAS_LIMIT ] (reason="execution reverted", method="estimateGas", transaction={"from":"0x9631288F4050F7CFbf77B77f8540DeCF6cfC7012","to":"0xdAC17F958D2ee523a2206206994597C13D831ec7","data":"0x095ea7b30000000000000000000000003ee18b2214aff97000d974cf647e7c347e8fa58500000000000000000000000000000000000000000000000000000000000f4240","accessList":null}, error={"code":-32000,"message":"execution reverted"}, code=UNPREDICTABLE_GAS_LIMIT, version=providers/5.7.2)
    at Logger.makeError (main.js:166:46729)
    at Logger.throwError (main.js:166:46862)
    at checkError$1 (main.js:172:114180)
    at Web3Provider.<anonymous> (main.js:172:124788)
    at Generator.throw (<anonymous>)
    at mn (main.js:172:113201)

image

AigaD commented 8 months ago

Tested from: https://wormhole-connect-mainnet.netlify.app/

Also reproduced when sending USDT from Ethereum (Metamask) to Arbitrum). Transaction fails with reason Error: cannot estimate gas; transaction may fail or may require manual gas limit

Attachment:

Screenshot 2024-03-18 at 10 37 03
tsadovska commented 7 months ago

Issue is reproducible on https://preview.portalbridge.com/b05f4e9322/ [Mainnet] with following scenario:

  1. Prepare to send 2 USDT from Ethereum (MetaMask) to BSC (MetaMask)
  2. Select "Automatic Bridge" option in Route section
  3. Change slider position
  4. Click on "Approve and proceed with transaction" button
Error: cannot estimate gas; transaction may fail or may require manual gas limit [ See: https://links.ethers.org/v5-errors-UNPREDICTABLE_GAS_LIMIT ] (reason="execution reverted", method="estimateGas", transaction={"from":"0x9631288F4050F7CFbf77B77f8540DeCF6cfC7012","to":"0xdAC17F958D2ee523a2206206994597C13D831ec7","data":"0x095ea7b3000000000000000000000000cafd2f0a35a4459fa40c0517e17e6fa2939441ca00000000000000000000000000000000000000000000000000000000001e8480","accessList":null}, error={"code":-32000,"message":"execution reverted"}, code=UNPREDICTABLE_GAS_LIMIT, version=providers/5.7.2)
    at Logger.makeError (pb-index-DsRyVLaa.js:334:46729)
    at Logger.throwError (pb-index-DsRyVLaa.js:334:46862)
    at checkError$1 (pb-index-DsRyVLaa.js:340:114131)
    at Web3Provider.<anonymous> (pb-index-DsRyVLaa.js:340:124730)
    at Generator.throw (<anonymous>)
    at so (pb-index-DsRyVLaa.js:340:113152)

image

AigaD commented 7 months ago

Tested from: https://portalbridge.com/ Transaction 0.1 USDT from Ethereum (Metamask) to Osmosis (Leap) fails, reason Error: cannot estimate gas; transaction may fail or may require manual gas limit Attachment:

Screenshot 2024-04-08 at 11 35 55
AigaD commented 6 months ago

Tested from: Wormhole Mainnet Transaction USDT from Ethereum (Metamask) to Polygon (Metamask) tested OK Transaction USDT from Ethereum (Metamask) to Arbitrum (Metamask) tested OK Tested from: https://portalbridge.com/ Transaction USDT from Ethereum (Metamask) to Osmosis (Leap) tested OK Closing the ticket.

Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/7c42f66a-c846-436c-aacd-c15da537fc94