search

brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.57k stars 2.28k forks source link

Brave browser doesn't imports data from MetaMask and even corrupts it #7860

Closed danimesq closed 4 years ago

danimesq commented 4 years ago

Data in MetaMask was reset and extension corrupted after the update adding CryptoWallets. brave corrupts metamask without importing its data

danimesq commented 4 years ago

There was no responsibility when releasing this update.

srirambv commented 4 years ago

@DaniellMesquita I've requested in almost every issue you have opened to please use the template but you ignore it. It's impossible to debug if you just say data is corrupted. Please edit the issue and add the details that are part of the template so that we can figure out whats the issue. Will have to close the issue for lack of information (already mentioned in the template header)

danimesq commented 4 years ago

To reproduce:

  1. I had a previous version with MetaMask
  2. It was automatically updated to version 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit)
  3. I'd noticed it came with its own fork of MetaMask (CryptoWallets) and the two were conflicting when web3 sites did requests. Then MetaMask got corrupted with its data.
srirambv commented 4 years ago

Will be fixed with https://github.com/brave/brave-browser/issues/7503

danimesq commented 4 years ago

I've requested in almost every issue you have opened to please use the template but you ignore it

About ignoring, re-open this issue.

Will be fixed with #7503

But isn't yet. Also, will this fix recover corrupted data from MetaMask? Nope. A browser update corrupting data is a immature and irresponsible act from a development team. We should talk instead of attacks.