568136 - Migration requires client to check for a node again after a 401.

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[Zandr Milewski [:zandr]]

When we migrate users we change their primaryNode in LDAP. This means that sync attempts against the old node will 401.

Disconnect/Reconnect is sufficient to make things work again, but this needs to be seamless.

Per mconnor, right solution it to catch a 401 during sync and log out/log in.

Comment 1

[Ed Lee :Mardak]

Attachment: v1

Comment 2

[Mike Connor [:mconnor]]

Comment on attachment 449144 [details] [diff] [review]
v1

other places to deal with off the top of my head:

* heartbeat
* fx tabs page
* fennec tabs page

Comment 3

[Ed Lee :Mardak]

Attachment: v2

Components.utils.import("resource://weave/service.js"); Weave.Service.clusterURL = "https://sj-weave06.services.mozilla.com/";

doing a sync:

2010-06-03 17:41:57	Service.Main         DEBUG	Caching URLs under storage user base: https://sj-weave06.services.mozilla.com/1.0/edtest/
2010-06-03 17:42:02	Net.Resource         DEBUG	GET fail 401 https://sj-weave06.services.mozilla.com/1.0/edtest/info/collections
2010-06-03 17:42:02	Service.Main         DEBUG	Finding cluster for user edtest
2010-06-03 17:42:03	Net.Resource         DEBUG	GET success 200 https://auth.services.mozilla.com/user/1.0/edtest/node/weave
2010-06-03 17:42:03	Service.Main         DEBUG	cluster value = https://sj-weave07.services.mozilla.com/
2010-06-03 17:42:03	Service.Main         DEBUG	Caching URLs under storage user base: https://sj-weave07.services.mozilla.com/1.0/edtest/
2010-06-03 17:42:03	Net.Resource         DEBUG	GET success 200 https://sj-weave07.services.mozilla.com/1.0/edtest/info/collections

Comment 4

[Ed Lee :Mardak]

Attachment: v2.1

Comment 5

[Mike Connor [:mconnor]]

Comment on attachment 449163 [details] [diff] [review]
v2.1

yeah, that's good.  doing our own URL munging makes me twitchy, but I think it's okay and necessary here.

Comment 6

[Ed Lee :Mardak]

http://hg.mozilla.org/labs/weave/rev/5346547346ae
Allow observers to watch for resource 401s and to specify a new uri to request. Test to make sure "weave:resource:status:401" gets notified and resource handles uri changes.

Comment 7

[Ed Lee :Mardak]

For 1.3.x branch

http://hg.mozilla.org/labs/weave/rev/77c464bd3f3a
Allow observers to watch for resource 401s and to specify a new uri to request.