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]: Huge number of accounts generated within the wallet when utilizing development builds ('Profile sync' is ON) #27395

Open
sleepytanya opened this issue Sep 25, 2024 · 1 comment
Labels
area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. regression-develop Regression bug that was found on development branch, but not yet present in production Sev2-normal Normal severity; minor loss of service or inconvenience. team-accounts team-notifications Notifications team type-bug

Comments

@sleepytanya
Copy link
Contributor

sleepytanya commented Sep 25, 2024

Describe the bug

Slack discussion link https://consensys.slack.com/archives/CBHSSFEN5/p1727135446963979

Big number of accounts generated within the wallet on development builds leading to a noticeable decrease in performance, rendering the wallet unusable.

"With profile sync on, the builds using develop will indeed either:
save the accounts info to the user storage if it never happened before
retrieve accounts from user storage if present, and add new accounts until we locally reach the same length as present in user storage "( thank you, Mathieu!)

Not sure this is in the scope of this ticket but this is 'broken' account https://sepolia.etherscan.io/address/0xA24B5c474dF517b083811b099054D670Ba696C7a
Last transaction created with the nonce 2102, while previous transaction nonce is 325.

Expected behavior

Screenshots/Recordings

100+ accounts - recording is redacted, the onboarding took ~ 4 mins:

hundredAccounts.mov

Same number of accounts (117) both in the UI and in the JSON response (hashedkKey):

Screen.Recording.2024-09-25.at.10.00.03.mov

Screenshot 2024-09-25 at 10 14 31

Steps to reproduce

  1. Set Settings -> Security & Privacy -> Privacy -> Profile sync ON

Error messages or log output

No response

Detection stage

On the development branch

Version

12.2.0

Build type

None

Browser

Chrome, Firefox

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

Severity

No response

@sleepytanya sleepytanya added type-bug team-accounts regression-develop Regression bug that was found on development branch, but not yet present in production labels Sep 25, 2024
@Prithpal-Sooriya Prithpal-Sooriya added the team-notifications Notifications team label Sep 25, 2024
@hjetpoluru hjetpoluru added the Sev2-normal Normal severity; minor loss of service or inconvenience. label Sep 26, 2024
@seaona seaona added the area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. label Sep 27, 2024
@sleepytanya
Copy link
Contributor Author

sleepytanya commented Oct 3, 2024

Encountered the same issue with another wallet on the latest develop again, extension is stuck on loading screen, new error message:

Screenshot 2024-10-02 at 20 36 59

On the latest develop every time account is imported the amount of accounts is higher (almost two times higher).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-performance Issues relating to slowness of app, cpu usage, and/or blank screens. regression-develop Regression bug that was found on development branch, but not yet present in production Sev2-normal Normal severity; minor loss of service or inconvenience. team-accounts team-notifications Notifications team type-bug
Projects
Status: To be fixed
Status: To be fixed
Status: To be fixed
Development

No branches or pull requests

4 participants