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

CryptoWallet: Imported ETH private key; deleted wallet; still available in "Transfer Between my Accounts" #8823

Open
heyJonBray opened this issue Mar 23, 2020 · 0 comments
Assignees

Comments

@heyJonBray
Copy link

@heyJonBray heyJonBray commented Mar 23, 2020

Description

I was playing around with crypto wallet features to import 3rd party wallets from the private key. This worked great, all of my ERC20 token balances showed up fine. I removed the account from my Crypto Wallets, but when I open my Brave-created or MetaMask wallet and send funds, my third wallet is still shown.

Steps to Reproduce

  1. brave://wallet > Import Account > Private Key
  2. Enter private key for ETH wallet
  3. Remove connected wallet.
  4. Open any other wallet and select 'Send' > 'Transfer Between my Accounts'
  5. The deleted wallet is still shown there, although without the name it was given, just "Address 3".

Actual result:

A wallet which contained my private key is still showing up as a contact even after removing.

Expected result:

I don't want to see that wallet anymore after removing.

Reproduces how often:

Easily reproduced. Tried with Exodus and MyEtherWallet.

Brave version (brave://version info)

Brave 1.5.113 Chromium: 80.0.3987.149 (Official Build) (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS Windows 10 OS Version 1909 (Build 18363.720)

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
    No
  • Does the issue resolve itself when disabling Brave Rewards?
    No
@bbondy bbondy added this to Untriaged in Crypto Wallets via automation Mar 27, 2020
@bbondy bbondy moved this from Untriaged to Backlog in Crypto Wallets Mar 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Crypto Wallets
  
Backlog
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.