#1227 Improvement for X509CertImpl.
Closed: migrated 3 years ago by dmoluguw. Opened 9 years ago by edewata.

Currently the X509CertImpl constructor takes a byte array of certificate data, so in many cases the certificate objects obtained from JSS have to be explicitly encoded first:

X509Certificate jssCert = cryptoManager.findCertByNickname(nickname);
byte[] certData = jssCert .getEncoded();
X509CertImpl cert = new X509CertImpl(certData);

It would be nice to add another constructor that can take the JSS certificate directly:

X509Certificate jssCert = cryptoManager.findCertByNickname(nickname);
X509CertImpl cert = new X509CertImpl(jssCert);

Internally the constructor will simply get the encoded certificate data from the parameter, then pass it to the existing X509CertImpl constructor that takes the encoded certificate data.

The constructor could also check if the parameter is already an instance of X509CertImpl. In that case it can do a normal Java copy constructor operation, but we should consider if it's much more efficient than encoding and decoding the certificate data.


Per CS/DS meeting of 12/15/2014: Milestone 10.3

Metadata Update from @edewata:
- Issue set to the milestone: UNTRIAGED

7 years ago

Dogtag PKI is moving from Pagure issues to GitHub issues. This means that existing or new
issues will be reported and tracked through Dogtag PKI's GitHub Issue tracker.

This issue has been cloned to GitHub and is available here:
https://github.com/dogtagpki/pki/issues/1789

If you want to receive further updates on the issue, please navigate to the
GitHub issue and click on Subscribe button.

Thank you for understanding, and we apologize for any inconvenience.

Metadata Update from @dmoluguw:
- Issue close_status updated to: migrated
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata