[Bug]: ETH Mainnet - Unable to deposit to smart contract.

[catsmells]

Describe the bug

When attempting to deposit ETH to a deployed smart contract, MetaMask grays out the ability to create the transaction despite ample balance for the amount being sent, and the gas itself.

Expected behavior

No response

Screenshots/Recordings

https://recordit.co/mhS6eX9NmU

Steps to reproduce

1. Copy deployed smart contract address (deployed from Trezor-Metamask Bridge).
2. Paste it in "Send".
3. Specify amount of ETH you wish to send.

Error messages or log output

No response

Version

11.5.1

Build type

None

Browser

Firefox

Operating system

Windows

Hardware wallet

Trezor

Additional context

No response

Severity

No response

[catsmells]

Hello @catsmells, Visit the Ethereum official page to resolve this issue.

Note: you can also initiate a chat with the support for more assistance and enquires.

That's a phishing link. Reported.
The actual ETH link: https://ethereum.org/en/community/support/

[bschorchit]

Thank you for your report @catsmells ! We'll look into it. I'm able to reproduce it as well on v11.6.2 and v11.7

[seaona]

does this contract have a receive function @metamaskbot ? I've encountered this behaviour with contracts which don't have receive function, but I can normally send ETH to contract addresses which have receive.

Example:
Sending ETH to this address works normally:

contract Test {
   receive() external payable { }    
}

but sending ETH to this contract does not work:

contract Test {}

The gas is set to 0 and cannot proceed to the next screen (similarly to your recording). I see the error Error#6: Unable to determine contract standard in the console`. This closely relates to this other ticket #21867 and we need a way to better handle those cases.

cc @bschorchit

[catsmells]

does this contract have a receive function @metamaskbot ? I've encountered this behaviour with contracts which don't have receive function, but I can normally send ETH to contract addresses which have receive.

Example: Sending ETH to this address works normally:

contract Test {
   receive() external payable { }    
}

but sending ETH to this contract does not work:

contract Test {}

The gas is set to 0 and cannot proceed to the next screen (similarly to your recording). I see the error Error#6: Unable to determine contract standard in the console`. This closely relates to this other ticket #21867 and we need a way to better handle those cases.

cc @bschorchit

Interesting tidbit - contracts without the receive function allow for deposits on testnets, but not on the Mainnet.

[blackdevelopa]

Also worth mentioning that MMM handles this differently (gets to the confirm screen) and that both platforms should be consolidated.

@seaona shared two ideas around this.

1. We should allow to proceed with the flow in Extension as is on mobile (we should not censor anything, and we must empower the user to decide)
2. We should display a warning for failing transactions in both Mobile and Extension.

@bschorchit