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

ETH swap from Wallet botched, out of gas, over $100 fee never refunded #19804

Closed
XfeRecover opened this issue Nov 29, 2021 · 2 comments
Closed
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/web3/wallet/core feature/web3/wallet Integrating Ethereum+ wallet support needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop priority/P2 A bad problem. We might uplift this to the next planned release.

Comments

@XfeRecover
Copy link

Description

Yesterday I did my first swap with the new integrated Brave wallet, from ETH to USDC. The quoted fee was outrageous ($130), but that's not Brave's fault.

The problem is that the transfer ran "out of gas". Here is the tx.

I don't see any way in Brave to complete or speed up the transfer.

Brave didn't warn me about the possibility of insufficient gas. This issue cost me $121 (as of today). In my country, that is food for two weeks.

I'm not a super-technical user, so I don't really understand all the inner workings of gas. If this occurrence is likely, there should be a big red warning before users attempt swaps (or presumably transfers as well).

I would also appreciate a refund of the fee.

Steps to Reproduce

As you can imagine, I won't attempt to reproduce this.

Brave version (brave://version info)

Brave 1.32.113 Chromium: 96.0.4664.45 (Official Build) (64-bit)
Revision 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS Linux

Version/Channel Information:

Version 1.32.113 Chromium: 96.0.4664.45 (Official Build) (64-bit)

@jamesmudgett jamesmudgett self-assigned this Nov 30, 2021
@jamesmudgett jamesmudgett added the feature/web3/wallet Integrating Ethereum+ wallet support label Nov 30, 2021
@jamesmudgett
Copy link

Needs UI follow up, self-assigning

@bbondy bbondy added the priority/P2 A bad problem. We might uplift this to the next planned release. label Nov 30, 2021
@bbondy bbondy added the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Dec 16, 2021
@bbondy bbondy added the needs-more-info The report requires more detail before we can decide what to do with this issue. label Mar 17, 2022
@bbondy bbondy added this to Web3 Mar 17, 2022
@bbondy bbondy moved this to Backlog in Web3 Mar 17, 2022
@jamesmudgett
Copy link

We've added CoinGecko price difference label to the swap quotes in swap 2.0, currently in Beta. This will be released to production in 1.48 on February, 7 2023

@github-project-automation github-project-automation bot moved this from Backlog to Done in Web3 Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/web3/wallet/core feature/web3/wallet Integrating Ethereum+ wallet support needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop priority/P2 A bad problem. We might uplift this to the next planned release.
Projects
Archived in project
Development

No branches or pull requests

4 participants