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

Enable registry for ACME #4860

Merged
merged 1 commit into from
Sep 26, 2024
Merged

Enable registry for ACME #4860

merged 1 commit into from
Sep 26, 2024

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Sep 26, 2024

The default pki_registry_enable for ACME has been changed to True. This allows pkispawn and pkidestroy to create and remove ACME properly.

The PKIDeployer.create_acme_subsystem() has been modified to create the registry. The remove_acme_subsystem() has been modified to remove the registry.

The PKISubsystem.remove_registry() has been modified to check whether the files/folders exist before removing them in case the subsystem was created without registry.

The test for ACME on separate instance has been modified to no longer create the server and NSS database before calling pkispawn. Some file/folder permissions have also changed due to these changes.

The default pki_registry_enable for ACME has been changed
to True. This allows pkispawn and pkidestroy to create and
remove ACME properly.

The PKIDeployer.create_acme_subsystem() has been modified
to create the registry. The remove_acme_subsystem() has
been modified to remove the registry.

The PKISubsystem.remove_registry() has been modified to
check whether the files/folders exist before removing them
in case the subsystem was created without registry.

The test for ACME on separate instance has been modified to
no longer create the server and NSS database before calling
pkispawn. Some file/folder permissions have also changed due
to these changes.
Copy link

sonarcloud bot commented Sep 26, 2024

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 Sep 26, 2024

@fmarco76 Thanks!

@edewata edewata merged commit 5fa0ac4 into dogtagpki:master Sep 26, 2024
156 of 164 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