-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[Security] Sync security improvements: Delete account #22884
Labels
feature/sync
OS/Android
Fixes related to Android browser functionality
OS/Desktop
QA Pass - Android ARM
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Test-Plan-Specified
QA/Yes
release-notes/include
security
Milestone
Comments
AlexeyBarabash
added
feature/sync
OS/Android
Fixes related to Android browser functionality
OS/Desktop
labels
May 17, 2022
25 tasks
LaurenWags
changed the title
Sync security improvements: Delete account
[Security] Sync security improvements: Delete account
Jan 13, 2023
stephendonner
added
the
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
label
Jan 13, 2023
Verification
|
Brave | 1.48.125 Chromium: 109.0.5414.87 (Official Build) beta (x86_64) |
---|---|
Revision | 2dc18eb511c56e012081b4abc9e38c81c885f7d4-refs/branch-heads/5414@{#1241} |
OS | macOS Version 11.7.2 (Build 20G1020) |
Case A: Initial case - PASSED
- installed
1.48.125
- launched Brave
- created the sync chain between desktop
deviceA
(macOS
) anddeviceB
(Win 10
) - kept the Sync settings page opened on both A and B
- on
deviceA
, pressed theDelete Sync account
button - confirmed
deviceB
's sync state was also reset - confirmed I did not see the informer
deviceA |
deviceB |
deviceA |
deviceA |
deviceB |
---|---|---|---|---|
Case B: Infobar on 2nd device - PASSED
- new profile
- create the sync chain
- on
deviceB
: close tabbrave://settings/braveSync/setup
, close browser app - on
deviceA
, pressedDelete Sync account
, agreed with warning - opened browser on
deviceB
Confirmed I saw the info bar on deviceB
Confirmed re-create the account
opened brave://settings/BraveSync/setup
in a new tab
Confirmed OK
dismissed the infobar
Confirmed Sync data/state on deviceB
was wiped/reset
deviceA |
deviceB |
deviceA |
deviceA |
deviceB |
deviceB |
---|---|---|---|---|---|
Case C: Deny entry to deleted sync chains - PASSED
- new profile
- created a sync chain
- copied the sync code to the clipboard
- deleted the sync account
- tried to re-enter the chain with the copied sync words
Confirmed I saw the Could not join this sync chain. Account was deleted.
error message
example | example | example | example | example | example | example |
---|---|---|---|---|---|---|
Case D: Upgrade Delete Sync Account
- PASSED
- install
1.47.177
ondeviceA
- create a sync chain for desktop
deviceA
() anddeviceB
() - close Brave for
deviceA
- launch Brave with
1.48.137
fordeviceA
- click
Delete Sync Account
- clicked
Permanently delete account
- confirmed
deviceB's
sync state was also reset - confirmed no message is shown on
deviceB
deviceA |
deviceB |
deviceA (1.47.177 ) |
deviceA (1.48.137 ) |
deviceA |
deviceB |
---|---|---|---|---|---|
Case E: Upgrade infobar on 2nd device - PASSED
stephendonner
added
QA Pass-macOS
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Jan 13, 2023
MadhaviSeelam
added
the
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
label
Jan 18, 2023
Verification
New InstallCase 1: Initial case -
|
deviceA |
deviceB |
deviceA |
deviceA |
deviceA |
deviceB |
---|---|---|---|---|---|
Case 2: Infobar on 2nd device - re-create the account
- PASSED
- new profile
- create a sync chain in
deviceA
(Linux
) anddeviceB
(Win 11 x64
) - on
deviceA
: close tabbrave://settings/braveSync/setup
, close browser app - click
Delete Sync Account
ondevice B
- clicked
Permanently delete account
- opened browser on
deviceA
-
Confirmed I saw the info bar on
deviceA
-
Confirmed
re-create the account
openedbrave://settings/BraveSync/setup
in a new tab -
Confirmed
OK
dismissed the infobar -
Confirmed Sync data/state on
deviceA
was wiped/reset
deviceA |
deviceB |
deviceB |
deviceB |
deviceA |
deviceA |
---|---|---|---|---|---|
Case 3: Deny entry to deleted sync chains - Could not join this sync chain
message - PASSED
- new profile
- created a sync chain
Profile 1
andProfile 2
- copied the sync code to the clipboard
- deleted the sync account
- tried to re-enter the chain with the copied sync words
Confirmed I saw the Could not join this sync chain. Account was deleted.
error message
Profile 1 | Profile 2 | example | example | example | example |
---|---|---|---|---|---|
Upgrade
Case 1: Upgrade - Delete Sync Account - PASSED
- Install
1.47.177
fordevice A
- install 1.48.132
for
device B` - create a sync chain for desktop
deviceA
(Linux
) anddeviceB
(Win 11 x64
) - on
device A
: close tabbrave://settings/braveSync/setup
, close browser app - re-launch Brave with
1.48.132
fordevice A
- kept the Sync settings page opened on both A and B
- click
Delete Sync Account
- clicked
Permanently delete account
- confirmed
deviceB
's sync state was also reset - confirmed no message is shown on
device B
step 1 | step 2 | step 3a | step3b | step 5 | step 8 | step 9 | step 10 |
---|---|---|---|---|---|---|---|
Case 2: Infobar on 2nd device - re-create the account
- PASSED
- Install
1.47.177
fordevice A
- install 1.48.132
for
device B` - create a sync chain for desktop
deviceA
(Linux
) anddeviceB
(Win 11 x64
) - on
deviceA
: close tabbrave://settings/braveSync/setup
, close browser app - click
Delete Sync Account
ondevice B
- clicked
Permanently delete account
- confirmed
deviceB
's sync state was reset - re-launch Brave with
1.48.132
fordevice A
-
Confirmed I saw the info bar on
deviceA
-
Confirmed
re-create the account
openedbrave://settings/BraveSync/setup
in a new tab -
Confirmed
OK
dismissed the infobar -
Confirmed Sync data/state on
deviceA
was wiped/reset
step 1 | step 2 | step 3a | step 3b | step 6 | step 7 | step 8 | result |
---|---|---|---|---|---|---|---|
MadhaviSeelam
added
QA Pass-Win64
QA Pass-Linux
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Jan 23, 2023
stephendonner
added
the
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
label
Jan 26, 2023
Verification
|
example | example | example | example | example | example |
---|---|---|---|---|---|
Case B
- Android can't enter deleted sync chain with code words - PASSED
- created the sync chain on Android
- copied sync words
- tapped
Delete Sync account
and confirmed - tried to
Join the sync chain
with the code words step 2
Confirmed error message: Could not join this sync chain. Account was deleted.
example | example | example | example | example | example |
---|---|---|---|---|---|
Case C
- Android can't enter deleted sync account with QR code - PASSED
- created the sync chain on Desktop
- displayed the QR code and took a screenshot
- deleted the sync chain from Desktop
- on Android, tried to enter the sync chain by scanning the QR code in the screenshot
Confirmed error message: Could not join this sync chain. Account was deleted.
example | example | example | example | example | example | example |
---|---|---|---|---|---|---|
Case D
- Android shows informer when its sync chain is deleted from another device - PASSED
- created the sync chain on Desktop
Joined the sync chain
from Android- quit browser app on Android
- deleted the sync chain from Desktop browser
- launched browser on Android
- expected to see the informer:
This Brave Sync account was deleted from other device. If you didn't mean to remove the Sync, re-create the account
- tapped
re-create the account
- expected to be on Sync Setup screen
Confirmed re-create the account
informer appears and goes Brave Sync
setup screen
Confirmed tapping OK
dismisses the dialog
Desktop
example | example | example | example | example | example | example |
---|---|---|---|---|---|---|
Android
example | example | example | example | example |
---|---|---|---|---|
stephendonner
added
QA/Test-Plan-Specified
QA Pass - Android ARM
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Jan 26, 2023
This was referenced Feb 7, 2023
I tried that option a couple of times (because problem with syncing devices), and I keep on getting stuck at "Account deletion is in progress" |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/sync
OS/Android
Fixes related to Android browser functionality
OS/Desktop
QA Pass - Android ARM
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Test-Plan-Specified
QA/Yes
release-notes/include
security
A user who wants to stop using Sync should have the ability to delete their encrypted data from our server. Next to the “Leave Sync chain” button at the bottom of the page, we should have a “Delete Sync account” button. It would trigger the following confirmation dialog:
Spec: https://docs.google.com/document/d/1X7glDHmeIjgipcPWn__S9rSxrI39P9rNw0RmuEuC3BI/edit#heading=h.f9zbg4b885l
The text was updated successfully, but these errors were encountered: