Closed
Bug 710543
Opened 13 years ago
Closed 12 years ago
Bring back 'change.password.pwSameAsRecoveryKey' string
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: philikon, Assigned: raymondlee)
References
Details
(Keywords: regression, Whiteboard: [good first bug][lang=js][mentor=philikon])
Attachments
(1 file, 1 obsolete file)
1.24 KB,
patch
|
Details | Diff | Splinter Review |
Bug 675823, specifically part 4 (see bug 675823 comment 17), removed the 'change.password.pwSameAsRecoveryKey', but it's still used in the password change dialog (via syncUtils.js) Bug 689832 comment 6 even claims that this will let you set your password to the Sync Key. I'm skeptical that this is the case, but I haven't verified.
Reporter | ||
Comment 1•13 years ago
|
||
Looping in Axel. The smallest fix here would be to simply bring back the string that was erroneously removed. Is that gonna be ok on Aurora? If not, we can also just get rid of the check entirely. I mean, we don't let users choose their own Sync Key anymore anyway. The whole "your password mustn't be equal to your passphrase" isn't really an issue anymore. If you want your password to be a 26 character long ass string, fine...
Comment 2•13 years ago
|
||
We're 5 weeks into the string freeze on aurora, we should get along without this string, if possible.
Reporter | ||
Comment 3•13 years ago
|
||
Then I propose we just rip out that functionality. rnewman, what do you think?
Comment 4•13 years ago
|
||
(In reply to Philipp von Weitershausen [:philikon] from comment #3) > Then I propose we just rip out that functionality. rnewman, what do you > think? I was just about to say! I think the original intent was to remove that check anyway. It pretty much stopped making sense once users could no longer pick a passphrase.
Reporter | ||
Updated•13 years ago
|
Whiteboard: [good first bug][lang=js][mentor=philikon]
Assignee | ||
Comment 5•12 years ago
|
||
Removed the check which uses "change.password.pwSameAsRecoveryKey".
Reporter | ||
Comment 6•12 years ago
|
||
Comment on attachment 623047 [details] [diff] [review] v1 403 gps
Attachment #623047 -
Flags: review?(philipp) → review?(gps)
Comment 7•12 years ago
|
||
Comment on attachment 623047 [details] [diff] [review] v1 Review of attachment 623047 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me. Raymond: Will you need someone to commit this for you? If so, please confirm the name and email address you would like reflected in the source control system and I will commit it for you.
Attachment #623047 -
Flags: review?(gps) → review+
Assignee | ||
Comment 8•12 years ago
|
||
Please you commit that for me. Thanks!
Attachment #623047 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 9•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e48be93aa253
Flags: in-testsuite-
Keywords: checkin-needed
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e48be93aa253
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Updated•6 years ago
|
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•