332608 - Propose to add a new function to delete certificate

Status: RESOLVED FIXED

(JSS Graveyard :: Library, defect)

Description

[Thomas Kwan]

Currently JSS has a function called PK11Store.deleteCert which delete the specified certificate and its key. We need to have a function to just delete certificate. We want to add

PK11Store.deleteCertOnly

Comment 1

[Thomas Kwan]

Attachment: New function PK11Store.deleteCertOnly

Comment 2

[glen beasley]

Comment on attachment 217070 [details] [diff] [review]
New function PK11Store.deleteCertOnly

looks good. Could you add a little javadoc to PK11Store.java for the the two methods:
for deleleCert state that is also deletes the key, and for deleteCertOnly state it only delete the cert.

Comment 3

[Thomas Kwan]

Attachment: New patch with comments

Glen, thanks for the quick turn around.

Comment 4

[Thomas Kwan]

Checked in to the trunk

Checking in lib/jss.def;
/cvsroot/mozilla/security/jss/lib/jss.def,v  <--  jss.def
new revision: 1.36; previous revision: 1.35
done
Checking in org/mozilla/jss/pkcs11/PK11Store.c;
/cvsroot/mozilla/security/jss/org/mozilla/jss/pkcs11/PK11Store.c,v  <--  PK11Store.c
new revision: 1.16; previous revision: 1.15
done
Checking in org/mozilla/jss/pkcs11/PK11Store.java;
/cvsroot/mozilla/security/jss/org/mozilla/jss/pkcs11/PK11Store.java,v  <--  PK11Store.java
new revision: 1.7; previous revision: 1.6
done