Closed Bug 1408492 Opened 7 years ago Closed 1 year ago

Move `MasterPassword.jsm` to Toolkit

Categories

(Toolkit :: Form Manager, enhancement, P3)

Product:
Toolkit
Component:
Form Manager
Type:
enhancement

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: lina, Unassigned)

References

Details

In bug 1388238, comment 0, MattN says: > For now we'll implement this in the formautofill system extension so we can iterate quickly but eventually this module can move to toolkit and be used by password manager and other consumers like sync. We'd like to replace Sync's bespoke, artisanal master password utility functions (https://searchfox.org/mozilla-central/rev/3a3af33f513071ea829debdfbc628caebcdf6996/services/sync/modules/util.js#475-508) with `MasterPassword.jsm`, so this bug tracks moving the JSM out of the extension and into Toolkit.
Priority: -- → P3
Severity: normal → S3

:lina is this still valid or we can close it?

Flags: needinfo?(lina)

Oh, wow, blast from the past! I did a bit of digging, and found that MasterPassword.jsm became OSKeyStore.jsm (and now OSKeyStore.sys.mjs) all the way back in bug 1486954, and its implementation changed to wrap the platform keystore instead of the Primary Password. Since that module doesn't exist anymore, I think we can go ahead and close this bug out. Thanks!

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: needinfo?(lina)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.