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

Fix pki-server subsystem-cert-find #4942

Merged
merged 1 commit into from
Jan 28, 2025
Merged

Fix pki-server subsystem-cert-find #4942

merged 1 commit into from
Jan 28, 2025

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jan 28, 2025

The PKISubsystem.get_cert_infos() has been updated to return a list instead of a generator object such that the number of entries can be counted using len().

The pki-server subsystem-cert-find/show commands have been deprecated since some of the certs returned by these commands are shared with other subsystems (e.g. sslserver, subsystem) which could cause some confusions. It's recommended to use pki-server cert-find/show instead.

The test for basic OCSP installation has been updated to check the above commands.

https://github.com/edewata/pki/blob/cli/docs/changes/v11.6.0/Tools-Changes.adoc#deprecate-pki-server-subsystem-cert-findshow

The PKISubsystem.get_cert_infos() has been updated to return
a list instead of a generator object such that the number of
entries can be counted using len().

The pki-server subsystem-cert-find/show commands have been
deprecated since some of the certs returned by these commands
are shared with other subsystems (e.g. sslserver, subsystem)
which could cause some confusions. It's recommended to use
pki-server cert-find/show instead.

The test for basic OCSP installation has been updated to
check the above commands.
@edewata edewata requested a review from fmarco76 January 28, 2025 01:23
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 Jan 28, 2025

@fmarco76 Thanks!

@edewata edewata merged commit 25cb21a into dogtagpki:master Jan 28, 2025
166 of 172 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