613617 - Addon UI: Update to simplify crypto

Status: VERIFIED FIXED

(Firefox :: Sync, defect)

Description

[Philipp von Weitershausen [:philikon]]

+++ This bug was initially created as a clone of Bug #612699 +++

This entails:

* Surface a notification for existing Sync users that a new Sync Key has been generated from their old Sync Key/passphrase. They don't need to do anything on their other machines, but their old Sync Key/passphrase is no longer valid.

* The account creation wizard won't allow you to set a custom passphrase, it just shows a generated one.

* The My Sync Key dialog no longer allows you to set a custom passphrase, it just allows you to generate a new one.

* Everywhere Sync Keys are shown and entered we need to aid the user with hyphens. The hyphenation/dehyphenation should happen in Weave.Util.* helpers, the UI code shouldn't have to know about specific Sync Key formats.

* Since we're ripping out custom passphrases we can also rip out the whole Sync
Key strength meter thingie. Yay!

Comment 1

[Philipp von Weitershausen [:philikon]]

Clearing blocking flag since this is for the add-on.

Comment 2

[Richard Newman [:rnewman]]

Attachment: Strength UI, v1

First stab at stripping out strength UI and tracking of "is this a custom key?".

This is a branch from Bug 603489's work.

Comment 3

[Richard Newman [:rnewman]]

16:43:26 <philiKON> the patch in Bug 597427 is now mostly obsolete because i'm revamping that code for jpake right now

Also adding a link to 'parent' bug.

Comment 4

[Philipp von Weitershausen [:philikon]]

(In reply to comment #3)
> 16:43:26 <philiKON> the patch in Bug 597427 is now mostly obsolete because i'm
> revamping that code for jpake right now

Bug 597427 is about the integrated Firefox UI, though, not the add-on UI (which is what this bug is about). So this and bug 597427 are quite unrelated.

Comment 5

[Richard Newman [:rnewman]]

> Bug 597427 is about the integrated Firefox UI, though, not the add-on UI (which
> is what this bug is about). So this and bug 597427 are quite unrelated.

Aaaah, I thought you were commenting on *my* patch, which is in this bug (Bug 613617). You're talking about your patch on Bug 597427, which I reviewed. 

Reading comprehension fail, sorry. Good to see this work still good, though :D

Comment 6

[Philipp von Weitershausen [:philikon]]

Attachment: v2

Rips out everything we don't need (strength meter, custom Sync Key, etc.). Hyphenates Sync Keys as you type (only needed when logging into an existing account and when your Sync Key was changed on another computer)

Comment 7

[Mike Connor [:mconnor]]

Comment on attachment 493067 [details] [diff] [review]
v2

>diff --git a/ui/firefox/locale/prefs.dtd b/ui/firefox/locale/prefs.dtd

> <!ENTITY syncKeyHelp.label         "What does the strength mean?">

Shouldn't this entity go as well?

Comment 8

[Philipp von Weitershausen [:philikon]]

(In reply to comment #7)
> > <!ENTITY syncKeyHelp.label         "What does the strength mean?">
> 
> Shouldn't this entity go as well?

Yes! Good catch.

Comment 9

[Philipp von Weitershausen [:philikon]]

Attachment: v3

Remove string + helper method for Sync Key help link as well.

Comment 10

[Philipp von Weitershausen [:philikon]]

https://hg.mozilla.org/services/fx-sync/rev/3641b7ea2565

Comment 11

[Tracy Walker [:tracy]]

verified with 1.6b1