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

"Error: cannot unlock without a previous vault" upon entering password #2011

Closed
rosiejr opened this issue Nov 24, 2020 · 18 comments · Fixed by #2034
Closed

"Error: cannot unlock without a previous vault" upon entering password #2011

rosiejr opened this issue Nov 24, 2020 · 18 comments · Fixed by #2034
Assignees
Labels
needs-reproduction Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing support type-bug Something isn't working

Comments

@rosiejr
Copy link

rosiejr commented Nov 24, 2020

Description
Upon closing the app, then reopening it to login, it rejects the user's password and shows the error, "Error: cannot unlock without a previous vault"

Screenshots

Error
Screenshot_20201115-200808

User's Device Info

Addtional error after uninstalling, reinstalling, disabled biometrics, closed the app, then reopening

To Reproduce
Steps to reproduce the behavior

  1. Close the mobile Android app
  2. Open it again and enter password
  3. Receive the error, "cannot restore without a previous vault"

@rickycodes has been able not been able to reliably reproduce this

Expected behavior
Users should be able to access their already imported wallet by entering their password

Smartphone

  • Device: v60 ThinQ
  • OS: Android
  • App Version 1.0.5

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

  • How critical is the impact of this bug on a user? Severe
  • Six customers have reported the issue
  • Is this visible to all users? No
  • Is this tech debt? No
@rosiejr rosiejr added the type-bug Something isn't working label Nov 24, 2020
@omnat omnat added the support label Nov 24, 2020
@rickycodes rickycodes self-assigned this Nov 24, 2020
@101001000
Copy link

This happened also to me today, metamask locked and I'm not able to login (today metamask has worked worse than habitual and then happened this)

@omnat
Copy link
Contributor

omnat commented Nov 25, 2020

@101001000 Sorry to hear, fixing this is our priority, and we are trying to reproduce it on our end to fix it. Are you able to fully quit the app and then login?
Could you also please share your device model, OS, and which app version you are on?

@omnat omnat added needs-reproduction Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing labels Nov 25, 2020
@omnat
Copy link
Contributor

omnat commented Nov 25, 2020

@101001000
Copy link

@101001000 Sorry to hear, fixing this is our priority, and we are trying to reproduce it on our end to fix it. Are you able to fully quit the app and then login?
Could you also please share your device model, OS, and which app version you are on?

Pocophone F1 Metamask 1.0.7 and android 9, I went back and now I can not even login, the button don't do anything

@randymetamask
Copy link

Another case: 62448

@101001000
Copy link

Idk what you can do to reproduce it, I'm a heavy user of the app and suddenly it happened. The same day I had more problems than usual to connect to web apps with metamask and the tokens balance weren't loading properly (but ether balance did), maybe that's relevant

@nottrunner
Copy link

nottrunner commented Nov 30, 2020

Having the same issue that started a couple days ago. Android 11, Pixel 4a.

image

@randymetamask
Copy link

In the mobile version a survey appeared, I completed it, then it asked me for the password to enter, and this error appears:

case67052

From case: 67052

@doruq
Copy link

doruq commented Dec 2, 2020

Guys how can we fix this? I can't use my mobile metamask for days because of this issue.

@101001000
Copy link

Same here, I'm waiting so I can use it again, is annoying

@0xWildhare
Copy link

This has been happening for me ever since the official version release on Android. worked fine before.

@mjgardner33
Copy link

Can someone help me. I'm having this problem, but don't know how to merge

@rosiejr
Copy link
Author

rosiejr commented Dec 7, 2020

Version 1.0.8 is live on Android. Please update your app and this issue should be resolved.

@101001000
Copy link

Version 1.0.8 is live on Android. Please update your app and this issue should be resolved.

The lastest release on GitHub and play store is 1.0.7 (I'm a beta tester)

@mjgardner33
Copy link

mjgardner33 commented Dec 7, 2020 via email

@mjgardner33
Copy link

Version 1.0.8 is live on Android. Please update your app and this issue should be resolved.

It corrupted my data. Thanks...

@rosiejr
Copy link
Author

rosiejr commented Dec 9, 2020

@mjgardner33 First, please check that you have your seed phrase and private keys you've imported to your MetaMask wallet. Next, uninstall and install the app again with the version 1.0.8. And import your seed phrase. Let us know if you continue to have an issue.

@101001000
Copy link

@mjgardner33 First, please check that you have your seed phrase and private keys you've imported to your MetaMask wallet. Next, uninstall and install the app again with the version 1.0.8. And import your seed phrase. Let us know if you continue to have an issue.

Without uninstalling and installing:
Uploading Screenshot_20201209-154010_MetaMask.png…

github-merge-queue bot pushed a commit that referenced this issue Dec 20, 2024
…2743)

## **Description**

Remove any usage of the global network from transaction simulation.

This requires using the `chainId` from the `TransactionMeta`, and so the
`SimulationDetails` properties have been simplified to accept
`TransactionMeta` directly to minimise the number of tightly coupled
properties.

## **Related issues**

Fixes: [#2011](MetaMask/mobile-planning#2011)

## **Manual testing steps**

## **Screenshots/Recordings**

### **Before**

### **After**

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-reproduction Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing support type-bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants