Can't log in a 2nd computer

RESOLVED FIXED in 1.4

Status

Product:
Cloud Services
Component:
Firefox Sync: Backend
RESOLVED FIXED
Reported:
8 years ago
Modified:
8 years ago

People

(Reporter: Mardak, Assigned: philikon)

Tracking

({regression})

Version:
unspecified
Keywords:
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [weave1.4])

Attachments

(1 attachment, 1 obsolete attachment)

Fix + test v1.1
2.58 KB, patch
Mardak
: review+
Details | Diff | Splinter Review
(Reporter)

Description

•
8 years ago 
Something regressed with bug 557590

2010-06-02 15:09:29	Service.Main         DEBUG	checkSetup: no username set
2010-06-02 15:09:49	Service.Main         DEBUG	Exception: aborting login, client not configured No traceback available

Comment 1

•
8 years ago 
Does this affect all 1.3 users? As in, if I am running 1.3, can I not set up a 2nd computer?
(Reporter)

Comment 2

•
8 years ago 
Nah. bug 557590 landed after 1.3.
 (Reporter)

Updated

•
8 years ago
Whiteboard: [weave1.4]
 (Assignee)

Comment 3

•
8 years ago 
Mea culpa. I was a bit overzealous with the refactoring and neglected that case :/. I'll prepare a patch.
Assignee: nobody → philipp
 (Assignee)

Comment 4

•
8 years ago 
Created attachment 449250 [details] [diff] [review]
Fix + test v1

Reintroduces an 'else' clause the refactoring in bug 557590 mistakenly threw out.
Attachment #449250 - Flags: review?(edilee)
 (Assignee)

Comment 5

•
8 years ago 
Created attachment 449260 [details] [diff] [review]
Fix + test v1.1

Improve teardown (clean up preferences).
Attachment #449250 - Attachment is obsolete: true
Attachment #449260 - Flags: review?(edilee)
Attachment #449250 - Flags: review?(edilee)
 (Reporter)

Comment 6

•
8 years ago 
Comment on attachment 449260 [details] [diff] [review]
Fix + test v1.1

>+    // We need a non-empty passphrase for login to work.  Even the
>+    // setup wizard just sets this to 'foo' if there isn't a
>+    // passphrase yet.
>+    Weave.Service.passphrase = "foo";
Can you file a bug to make sure this is investigated and made sure it's ok?
Attachment #449260 - Flags: review?(edilee) → review+
 (Reporter)

Comment 7

•
8 years ago 
http://hg.mozilla.org/labs/weave/rev/710dec717f8d
Fix a regression introduced with the status.js refactoring (bug 557590).
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.4
You need to log in before you can comment on or make changes to this bug.