Re: [Pki-devel] [PATCH] 528 Added CLI to import/export certificates with private keys.
On 10/9/2014 12:56 AM, Fraser Tweedale wrote:
Conditional ACK; one minor issue in command help. See inline.
> + option = new Option(null, "pkcs12", true,
"PKCS #12 file to store the certificate chain and the private key.");
> + option.setArgName("password");
Should be "path".
Fixed. I also fixed the other options and the man page for consistency.
On 10/9/2014 10:26 AM, Ade Lee wrote:
ACK
Something to think about, and which does not need to be part of this
patch.
Its possible that you might want to select a specific cert to import
from the p12 file - recall that that p12 file might contain multiple
certs, and that you would want to use a different nickname when saving
in the certDB. Offhand, I don't know how to do this using certutil and
p12util.
I opened a ticket for NSS tools:
https://bugzilla.redhat.com/show_bug.cgi?id=1151164
Thanks for the review. It's pushed to master with the above fixes.
--
Endi S. Dewata