Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Device removed from sync chain still receives new bookmarks/folders #952

Closed
srirambv opened this issue Mar 8, 2019 · 9 comments
Closed

Comments

@srirambv
Copy link
Contributor

srirambv commented Mar 8, 2019

Description:

Device removed from sync chain still receives new bookmarks/folders

Devices

Device 1: Ubuntu 18 running 0.61.47
Device 2: iPhone 7+ running 1.9(19.3.6.15)

Steps to Reproduce

  1. Add bookmarks and create sync chain on desktop
  2. Add iPhone to sync chain using QR code, ensure bookmarks sync
  3. Remove desktop from iPhone under devices, ensure desktop shows sync home page
  4. Add new bookmarks/folders on iPhone, after a while, new bookmarks/folders are sync'd on desktop as well

Actual result:
Desktop getting bookmarks after removed from sync chain
image
iOS sync chain with only one device
ezgif com-video-to-gif

Expected result:
Once device is removed any new bookmarks added should not be propagated to any device that was removed from the sync chain

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@srirambv
Copy link
Contributor Author

srirambv commented Mar 8, 2019

Any bookmark added on desktop gets sync'd to iOS device as well.

@iccub iccub self-assigned this Mar 8, 2019
@iccub
Copy link
Contributor

iccub commented Mar 12, 2019

Could you check if this happens on Android?

From my tests I noticed that the device doesn't get correctly deleted on desktop.
Look at SS below,

I synced 3 devices: iPhone, iPad, desktop
On iPad removed desktop, and as on the screenshot, it still shows sync menu with only iPhone and iPad

This still might indicate that our delete device command is wrong and doesn't work well with desktop

Zrzut ekranu 2019-03-12 o 18 29 40

@iccub
Copy link
Contributor

iccub commented Mar 12, 2019

Found this issue on desktop brave/brave-browser#3509

I think it needs a fix on desktop, not on iOS

@jhreis
Copy link
Contributor

jhreis commented Mar 20, 2019

This is certainly a desktop issue. It got the removal request from iOS since it updated the UI, but continues retrieving and sending bookmarks. Closing per @iccub's investigation.

@jhreis jhreis closed this as completed Mar 20, 2019
@GeetaSarvadnya
Copy link

The issue is reproducible on 1.9 (19.03.25.21)- iPhone8

@srirambv As per above comments issue is with Desktop, are you going to add an issue in bc desktop?

@iccub
Copy link
Contributor

iccub commented Mar 26, 2019

@GeetaSarvadnya it's logged here brave/brave-browser#3509

@srirambv
Copy link
Contributor Author

This issue needs to be reverified once brave/brave-browser#3509 is fixed.

cc: @brave/legacy_qa

@kjozwiak
Copy link
Member

Reproduced the original issue using the following builds:

  • 1.9 (19.3.27.19) on iPad Mini 4
  • 0.61.52 Chromium: 73.0.3683.86

I couldn't reproduce the issue using the following builds:

  • 1.9 (19.3.27.19) on iPad Mini 4
  • 0.64.22 Chromium: 73.0.3683.75

Went through the original STR three times using 0.64.22 Chromium: 73.0.3683.75 and waited ~30min. Once desktop was removed from the sync chain, it never received new changes as expected.

@kjozwiak
Copy link
Member

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

No branches or pull requests

5 participants