Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed JSSKeyStoreSpi.engineGetKey() #12

Merged
merged 1 commit into from
Jun 22, 2018
Merged

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jun 22, 2018

The JSSKeyStoreSpi.engineGetKey() has been rewritten in Java to
search for a private key by its cert. If that is not found, it
will search for a private key by its key ID. If that is not found
either, it will search for a symmetric key by its nickname.

https://pagure.io/jss/issue/10

The JSSKeyStoreSpi.engineGetKey() has been rewritten in Java to
search for a private key by its cert. If that is not found, it
will search for a private key by its key ID. If that is not found
either, it will search for a symmetric key by its nickname.

https://pagure.io/jss/issue/10
Copy link
Contributor

@jmagne jmagne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@edewata edewata merged commit 38c8e66 into dogtagpki:master Jun 22, 2018
@edewata edewata deleted the ticket-10 branch June 22, 2018 01:16
@edewata
Copy link
Contributor Author

edewata commented Jun 22, 2018

Thanks!

Merged.

@edewata edewata mentioned this pull request May 20, 2020
edewata added a commit to edewata/jss that referenced this pull request Jul 9, 2024
The CI tests has been updated to use pki nss-cert-import and
pki pkcs12-import commands to import certs and PKCS dogtagpki#12 files.
edewata added a commit to edewata/jss that referenced this pull request Jul 9, 2024
The CI tests has been updated to use pki nss-cert-import and
pki pkcs12-import commands to import certs and PKCS dogtagpki#12 files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants