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

Update pki nss-cert-import #4601

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Update pki nss-cert-import #4601

merged 1 commit into from
Nov 6, 2023

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Nov 3, 2023

The pki nss-cert-import has been updated to use the token name in the nickname if specified, otherwise it will use the token name specified in the --token option.

The NSSDatabase.addCertificate() in Java (which is used by pki nss-cert-import) has been modified to call the new PK11Store.importCert() instead of addPEMCertificate() which depends on certutil -A.

The NSSDatabase.add_cert() in Python has been updated to use JSS (via pki nss-cert-import) by default. It has also been updated to provide the input cert via standard input instead of file to avoid permission issues.

The pki nss-cert-import has been updated to use the token
name in the nickname if specified, otherwise it will use
the token name specified in the --token option.

The NSSDatabase.addCertificate() in Java (which is used
by pki nss-cert-import) has been modified to call the new
PK11Store.importCert() instead of addPEMCertificate()
which depends on certutil -A.

The NSSDatabase.add_cert() in Python has been updated to
use JSS (via pki nss-cert-import) by default. It has also
been updated to provide the input cert via standard input
instead of file to avoid permission issues.
@edewata edewata requested a review from fmarco76 November 3, 2023 18:22
Copy link

sonarqubecloud bot commented Nov 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

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

LGTM

@edewata
Copy link
Contributor Author

edewata commented Nov 6, 2023

@fmarco76 Thanks!

@edewata edewata merged commit 6dcca7e into dogtagpki:master Nov 6, 2023
132 checks passed
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