Closed
Bug 505401
Opened 15 years ago
Closed 15 years ago
Component should expose rewrapKey & verifyPassphrase methods
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
0.5
People
(Reporter: anant, Assigned: anant)
References
Details
Attachments
(1 file, 1 obsolete file)
11.08 KB,
patch
|
hello
:
review+
|
Details | Diff | Splinter Review |
The WrapPrivateKey and UnwrapPrivate key methods should be scriptable - this helps us re-encrypt a private key with a new passphrase from JS.
Comment 1•15 years ago
|
||
Perhaps a rewrap call with the current and new passphrases is better?
Assignee | ||
Comment 2•15 years ago
|
||
Yes! I think that is the best (or rather, the *only*) way to do this. NSS doesn't seem to allow export of unencrypted private keys :)
Assignee | ||
Updated•15 years ago
|
Summary: Component should expose RSA key methods → Component should expose an RSA key rewrap method
Assignee | ||
Comment 3•15 years ago
|
||
This function will do what we want. Unit test included.
Attachment #389622 -
Flags: review?(thunder)
Assignee | ||
Comment 4•15 years ago
|
||
While we're on the topic of changing crypto interfaces, I was thinking it will be nice to have a verifyPassphrase method that takes a key and passphrase and returns a boolean value. This will help us check for correctness of a user's passphrase before having to download the symmetric keys providing faster feedback.
Assignee | ||
Comment 5•15 years ago
|
||
Unit tests included.
Attachment #389622 -
Attachment is obsolete: true
Attachment #389722 -
Flags: review?(thunder)
Attachment #389622 -
Flags: review?(thunder)
Assignee | ||
Updated•15 years ago
|
Summary: Component should expose an RSA key rewrap method → Component should expose rewrapKey & verifyPassphrase methods
Comment 6•15 years ago
|
||
Comment on attachment 389722 [details] [diff] [review]
Adds a verifyPassphrase method in addition to rewrapPrivateKey
r=thunder
Attachment #389722 -
Flags: review?(thunder) → review+
Assignee | ||
Comment 7•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: Firefox Sync: Crypto → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•