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

sync stops working and password data loss, cr94 related #18138

Closed
rebron opened this issue Sep 15, 2021 · 3 comments · Fixed by brave/brave-core#10108
Closed

sync stops working and password data loss, cr94 related #18138

rebron opened this issue Sep 15, 2021 · 3 comments · Fixed by brave/brave-core#10108

Comments

@rebron
Copy link
Collaborator

rebron commented Sep 15, 2021

Description

On update to 1.31.41 chromium 94.0.4606.41, lost my passwords and noticed that sync is no longer working.

Steps to Reproduce

  1. Have sync enabled for several clients. Have passwords and other data types synced.
  2. Upgrade nightly from 1.31.x cr 93 version to 1.31.41 and above.
  3. Notice passwords are wiped out, sync internals aren't working, and brave://settings/braveSync/setup doesn't toggle.

Actual result:

Will provide sync-internals.

Expected result:

Sync should continue to be working and no data loss on passwords.

Reproduces how often:

Maybe difficult to reproduce.

Brave version (brave://version info)

This is arm64 macOS only. Windows continues to Sync and passwords are intact.
macOS 12.0 beta Monterey

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@rebron rebron added feature/sync OS/macOS priority/P1 A very extremely bad problem. We might push a hotfix for it. feature/password-manager OS/Desktop labels Sep 15, 2021
@AlexeyBarabash
Copy link
Contributor

Just reproduced this immediately, even after giving the keychain access

@AlexeyBarabash
Copy link
Contributor

cc @jsecretan , @darkdh

@stephendonner
Copy link

stephendonner commented Sep 17, 2021

Verified PASSED using

Brave 1.30.80 Chromium: 94.0.4606.50 (Official Build) beta (x86_64)
Revision 0c1ac2c4842a4746c27c937c1a0453f98da1a972-refs/branch-heads/4606@{#1049}
OS macOS Version 11.6 (Build 20G165)

Check keychain access

  1. Opened Keychain Access.app
  2. Under Local Items->All Items I found Chromium Safe Storage and Brave Safe Storage
  3. Opened them separately and delete all items in access control list
  4. Launched Brave with this fix (1.30.80)
  5. Brave should prompt asking for access to Brave Safe Storage instead of Chromium Safe Storage

Screen Shot 2021-09-17 at 11 09 05

Import

  1. Mdke sure Chrome had stored passwords
  2. Opened Brave and imported passwords from Chrome
  3. Allowed access to Chrome safe storage

Screen Shot 2021-09-17 at 11 10 41 AM

  1. Passwords were successfully imported
  2. change chrome to chromium and repeat step 1-4

Confirmed passwords were imported

example example example
Screen Shot 2021-09-17 at 11 10 38 AM Screen Shot 2021-09-17 at 12 16 59 PM Screen Shot 2021-09-17 at 12 16 27 PM

Migration

  1. Stored passwords, credit cards and setup sync in pre-C94 Brave (1.31.37)
  2. Upgraded to C94 Brave (1.31.41)
  3. Checked passwords, credit cards are gone and sync is not working
  4. Upgraded to Brave with this fix (1.30.80)
  5. Confirmed passwords & credit cards in step 1 are back, and sync works
example example example example
Screen Shot 2021-09-17 at 12 16 27 PM Screen Shot 2021-09-17 at 12 44 51 PM Screen Shot 2021-09-17 at 12 43 17 PM Screenshot_20210917-124033

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants