Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Cannot estimate gas fee on simple contract transaction #28356

Closed
buglov opened this issue Nov 7, 2024 · 5 comments
Closed

[Bug]: Cannot estimate gas fee on simple contract transaction #28356

buglov opened this issue Nov 7, 2024 · 5 comments
Labels
external-contributor regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 Sev1-high High severity; partial loss of service with severe impact upon users, with no workaround. team-confirmations Push issues to confirmations team type-bug

Comments

@buglov
Copy link

buglov commented Nov 7, 2024

Describe the bug

I have very simple contract with two functions whitelisteToken and removeWhitelistedToken:

mapping(address => bool) public whitelistedTokens;

function whitelistToken(address token) external {
    whitelistedTokens[token] = true;
}
function removeWhitelistedToken(address token) external {
    whitelistedTokens[token] = false;
}

When I try to execute a transaction, Metamask shows warning that cannot estimate the gas.

Expected behavior

Since the transaction is successful, I expect this warning to not be there and to estimate gas correctly

Screenshots/Recordings

Screenshot 2024-11-07 at 16 41 06 Screenshot 2024-11-07 at 16 27 53

Steps to reproduce

  1. Go to BSC Scan
  2. Connect your metamask
  3. Try to whitelistToken (for example 0x0000000000000000000000000000000000000000)

Error messages or log output

No response

Detection stage

During release testing

Version

12.6.0

Build type

None

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

Severity

No response

@buglov buglov added the type-bug label Nov 7, 2024
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by severity Nov 7, 2024
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by team Nov 7, 2024
@metamaskbot metamaskbot added external-contributor regression-RC-12.6.0 Regression bug that was found in release candidate (RC) for release 12.6.0 labels Nov 7, 2024
@DanielTech21 DanielTech21 added Sev1-high High severity; partial loss of service with severe impact upon users, with no workaround. team-confirmations Push issues to confirmations team labels Nov 7, 2024
@bschorchit
Copy link

Hey @buglov, thank you for your report! This will be fixed on our v12.6.1 release that should be rolling out soon

@vpintorico vpintorico added regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 and removed regression-RC-12.6.0 Regression bug that was found in release candidate (RC) for release 12.6.0 labels Nov 13, 2024
@bschorchit
Copy link

@buglov we have rolled out v12.6.1. Can you confirm if that your issue is fixed on this version?

@buglov
Copy link
Author

buglov commented Nov 14, 2024

@bschorchit tested this both on testnet and mainnet and its looks fixed! Thanks!

@bschorchit
Copy link

Thank you!

@github-project-automation github-project-automation bot moved this from To be fixed to Fixed in Bugs by severity Nov 16, 2024
@github-project-automation github-project-automation bot moved this from To be fixed to Fixed in Bugs by team Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contributor regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 Sev1-high High severity; partial loss of service with severe impact upon users, with no workaround. team-confirmations Push issues to confirmations team type-bug
Projects
Archived in project
Status: Fixed
Development

No branches or pull requests

6 participants
@buglov @vpintorico @bschorchit @metamaskbot @DanielTech21 and others