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

[Bug]: forced MM SRP backup when imported account balance is positive #9034

Closed
NicolasMassart opened this issue Mar 22, 2024 · 7 comments
Closed
Labels
regression-prod-7.18.1 Regression bug that was found in production in release 7.18.1 Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking stale Issues that have not had activity in the last 90 days team-wallet-ux type-bug Something isn't working

Comments

@NicolasMassart
Copy link
Contributor

NicolasMassart commented Mar 22, 2024

Describe the bug

MM displays a popup to force me to backup the hot MM SRP when I import an account that has positive balance.

  • As a user, I want to import accounts either form Ledger or private key.
  • When I setup a new MetaMask app, I create a wallet that I don't care about just to be able to reach the point where I can import my account.
  • I already have my private key or Ledger SRP properly backed-up and secured.
  • I don't care about the newly created MM hot account.
  • I have zero balance on the newly MM hot account.
  • I have positive balance on my account to import.
  • MM displays a popup that forces me to backup the hot MM wallet SRP that holds zero balance.

This is first annoying, but also confusing and dangerous as the user may misunderstand what SRP it is about, why we ask for this SRP to be backed-up when it actually has no balance, why they are told they have positive balance, ...

Note

It happens at least with Ledger and private key imports.
This is the exact same thing, none of PK or Ledger SRP is related to the newly generated SRP.

Expected behavior

MM should not force backup of SRP when the balance is positive on an imported account.

Screenshots/Recordings

Ledger import PK import
image image
screen-20240322-153548.mp4

Steps to reproduce

  1. Install fresh MM app
  2. create a new account
  3. do not backup SRP
  4. once in the wallet, import a new account from private key. Account must have non zero balance
  5. Popup that forces SRP backup appears and you can't escape until you backup the SRP that has no value on it.

Error messages or log output

No response

Version

7.18.1

Build type

None

Device

Moto Edge 30 Fusion

Operating system

Android

Additional context

No response

Severity

No response

@NicolasMassart NicolasMassart added the type-bug Something isn't working label Mar 22, 2024
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by severity Mar 22, 2024
@metamaskbot metamaskbot added the regression-prod-7.18.1 Regression bug that was found in production in release 7.18.1 label Mar 22, 2024
@NicolasMassart NicolasMassart added team-wallet-ux and removed regression-prod-7.18.1 Regression bug that was found in production in release 7.18.1 labels Mar 25, 2024
@NicolasMassart NicolasMassart changed the title [Bug]: forced MM SRP backup when Ledger balance is positive [Bug]: forced MM SRP backup when imported account balance is positive Mar 25, 2024
@metamaskbot metamaskbot added the regression-prod-7.18.1 Regression bug that was found in production in release 7.18.1 label Mar 25, 2024
@DanielTech21 DanielTech21 added the Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking label Mar 27, 2024
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by team Apr 9, 2024
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale Issues that have not had activity in the last 90 days label Aug 13, 2024
Copy link
Contributor

This issue was closed because there has been no follow activity in 7 days. If you feel this was closed in error please provide evidence on the current production app in a new issue or comment in the existing issue to a maintainer. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 20, 2024
@github-project-automation github-project-automation bot moved this from To be fixed to Fixed in Bugs by team Aug 20, 2024
@NicolasMassart NicolasMassart moved this from Fixed to To be fixed in Bugs by team Aug 26, 2024
@NicolasMassart
Copy link
Contributor Author

NicolasMassart commented Aug 26, 2024

This issue is still happening and I experienced it when testing #10727
I had zero balance on the network native coin but had some in MMGDAO tokens and it blocked me by forcing me to save the initial SRP that I don't care of.
This initial unsaved SRP was created only to be able to import a ledger account. I have no reason to backup it.

@NicolasMassart NicolasMassart removed the stale Issues that have not had activity in the last 90 days label Aug 26, 2024
@hesterbruikman
Copy link
Contributor

We'll need to think through the desired behavior here as the client isn't aware that you have no intention of using the newly created MM SRP. The issue seems to be that you require an MM SRP in the first place. @gantunesr Can you explain why currently an MM SRP is required before a private key or Ledger can be imported?

Copy link
Contributor

github-actions bot commented Dec 4, 2024

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale Issues that have not had activity in the last 90 days label Dec 4, 2024
@hesterbruikman hesterbruikman removed the stale Issues that have not had activity in the last 90 days label Dec 5, 2024
Copy link
Contributor

github-actions bot commented Mar 5, 2025

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale Issues that have not had activity in the last 90 days label Mar 5, 2025
Copy link
Contributor

This issue was closed because there has been no follow activity in 7 days. If you feel this was closed in error please provide evidence on the current production app in a new issue or comment in the existing issue to a maintainer. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-prod-7.18.1 Regression bug that was found in production in release 7.18.1 Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking stale Issues that have not had activity in the last 90 days team-wallet-ux type-bug Something isn't working
Projects
Archived in project
Status: To be triaged
Development

No branches or pull requests

4 participants