On Tue, Apr 11, 2017 at 03:23:18PM -0700, Christina Fu wrote:
Thank you. Please see review comments:
https://bugzilla.mozilla.org/show_bug.cgi?id=1355358#c6
I will review PKCS12Util later.
Christina
Updated patch jss-0002 and also created https://bugzilla.mozilla.org/show_bug.cgi?id=1359731 with some other JSS patches. Created Gerrit review branch for Dogtag patches: https://review.gerrithub.io/#/c/358634/. This includes patch pki-0178 and also a new patch to change KRA PKCS #12 recovery to use AES, which depends on the new JSS patches linked above. Thanks, Fraser
On 04/10/2017 11:30 PM, Fraser Tweedale wrote:
On Thu, Apr 06, 2017 at 03:45:55PM -0700, Christina Fu wrote:
Hi Fraser,
Could you please do the following first?
1. file a Mozilla bugzilla bug for this against Product JSS Release 4.4.1, then assign to yourself: https://bugzilla.mozilla.org/ 2. After making sure your patch compiles well with the 4.4.1 base, attach the patch to that ticket, and mark reviewers
thanks!
Christina
Thanks Christina, I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1355358
I was unable to assign myself to the bug ('Assignee' field is not active when I go to Edit Bug.
Also not sure how to "mark reviewers". I added you and Elio to Cc though.
Thanks, Fraser
On 04/04/2017 02:56 AM, Fraser Tweedale wrote:
Hi team,
Please review attached patches for JSS and Dogtag that:
- add some new EncryptedPrivateKeyInfo export and import functions to JSS
- update Dogtag's `pki pkcs12' command to use the new functions to achieve AES encryption of the key bags, with wrapping/unwrapping occurring on the token.
PKCS #12 files produced by current releases continue to import properly (of course, this is an important test vector).
These patches do not address the PKCS #12 KRA recovery export; This is my next task and separate patches will be produced.
Thanks, Fraser