Closed Bug 1298011 Opened 3 years ago Closed 3 years ago

Synced Tabs sidebar does not show a message to sign in again after password was changed

Categories

(Firefox :: Sync, defect, P3)

defect

Tracking

()

VERIFIED FIXED
Firefox 53
Tracking Status
firefox51 --- wontfix
firefox53 --- verified

People

(Reporter: cirdeiliviu, Assigned: eoger)

References

Details

(Whiteboard: [sync-quality])

Attachments

(2 files)

[Affected platforms]
all

[Steps to reproduce]:

1. Sign in to Firefox on 2 devices with the same account.
2. From one device open some tabs and perform a sync.
3. On second device open Sync Tabs sidebar (Menu Bar>View>Sidebars > Synced Tabs). > The tabs are listed in the Sync tabs sidebar.
4. On first device, go to your account and change your password. > Password in changed.
5. Perform a sync.
6. On your second device, on synced tabs sidebar right click and "Sync Now". 

[Expected Result]: 
Sync Tabs sidebar should show a message to sign in again.

[Actual Result]: 
Synced Tabs sidebar does not show a message to sign in again after password is changed. Closing and reopening the sidebar does not fix it.

[Regression range] 
I think this is not a regression, it works like this from the beginning of the implementation of the synced tabs sidebar.
Yeah, this sucks and isn't intentional :(
Liviu, could you please post a screenshot of what happens on the second device? What do you see instead of the message to sign in again?
Flags: needinfo?(liviu.cirdei)
Whiteboard: [sync-quality]
Attached video video_of_the_issue.mp4
The synced tabs and devices are displayed instead of the sign in message.
I attached a video that shows this.
Flags: needinfo?(liviu.cirdei)
From the triage meeting: there's some discussion around separating password change and deauthorizing a device, so we'll wait for that to crystallize before taking action.
Priority: -- → P3
This patch listens to the weave:service:login:change event which is fired when a password change has been detected.
Assignee: nobody → eoger
Status: NEW → ASSIGNED
Comment on attachment 8824219 [details]
Bug 1298011 - Update the Synced Tabs sidebar UI when account password changed.

https://reviewboard.mozilla.org/r/102730/#review103092

(oops - this was pending - apparently I forgot to hit "publish" :(

LGTM, thanks!
Attachment #8824219 - Flags: review?(markh) → review+
Pushed by eoger@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b6fcdb398694
Update the Synced Tabs sidebar UI when account password changed. r=markh
https://hg.mozilla.org/mozilla-central/rev/b6fcdb398694
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
We've built 51 RC. Mark 51 won't fix.
[bugday-20170222] bug verified 
os: Ubuntu 15.10
Browser Version: 41.0.2
I reproduced the bug and its fixed
Flags: qe-verify+
Kanchan, just making sure you're in the loop here.
Flags: needinfo?(kkumari)
Thanks Andrei!

I have tested this bug fix and found it to to be working fine on 
Version 	53.0b1
Build ID 	20170307064827

After password change on device 1, Synced Tabs sidebar UI on device 2 shows "sign in to sync" message.
Flags: needinfo?(kkumari)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.