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

Unable to execute transactions on Free Gas Networks #1890

Closed
lukeskywlkr opened this issue Oct 12, 2020 · 4 comments
Closed

Unable to execute transactions on Free Gas Networks #1890

lukeskywlkr opened this issue Oct 12, 2020 · 4 comments
Labels
community Issues or PRs opened by the MM community stale Issues that have not had activity in the last 90 days type-bug Something isn't working

Comments

@lukeskywlkr
Copy link

lukeskywlkr commented Oct 12, 2020

Describe the bug
The application is unable to execute transactions with gas price set to 0 on free gas networks. Metamask reports 0 as invalid number.

Screenshots
IMG_8030

To Reproduce
Steps to reproduce the behavior

  1. Connect to a Free Gas Network
  2. Execute a transaction and try to set the gas price to 0
  3. See error

Expected behavior
The app should allow the execution of a transaction on a free gas network and let the user modify accordingly the advanced transaction fee settings.

Smartphone (please complete the following information):

  • Device: iPhone6s
  • OS: iOS 14.0.1
  • App Version v1.0.3 (528)

to be added after bug submission by internal support / PM
Severity

  • How critical is the impact of this bug on a user?
  • Add stats if available on % of customers impacted
  • Is this visible to all users?
  • Is this tech debt?
@lukeskywlkr lukeskywlkr added the type-bug Something isn't working label Oct 12, 2020
@lukeskywlkr
Copy link
Author

Seems that the error is generated by this check at this line of code:

if (value === '0' || !value || value === '' || !isDecimal(value)) {

Could it be problematic to allow execution of transactions on such networks?

@ackhia
Copy link

ackhia commented Oct 27, 2020

Seems like this change was made in PR #1750 and has broken compatibility with Quorum and possibly other private network focused implementations.

@Obafs
Copy link

Obafs commented Jan 24, 2021

What advantage was obtained by making this change? In my current use, it prevents mobile users from accessing smart contract methods on private networks, that are available to the desktop version.

TL;DR: Please allow zero in custom gas price, unless there is a security reason not to.

@gantunesr gantunesr added the community Issues or PRs opened by the MM community label Sep 14, 2021
@sethkfman sethkfman added the stale Issues that have not had activity in the last 90 days label Jun 9, 2023
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 7 days with no activity. If you feel this was closed in error please reopen and provide evidence on the current production app. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Issues or PRs opened by the MM community stale Issues that have not had activity in the last 90 days type-bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants