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 server container #4754

Merged
merged 1 commit into from
May 28, 2024
Merged

Update PKI server container #4754

merged 1 commit into from
May 28, 2024

Conversation

edewata
Copy link
Contributor

@edewata edewata commented May 23, 2024

The PKI server container (which is the base for all PKI subsystem containers) has been updated to install the ROOT webapp (which provides Web UI files) and PKI webapp (which provides common PKI services), and store the default config files.

The startup script has been modified to install the default config files if it doesn't exist already, use PKI NSS CLI instead of certutil to prepare the certs, and run the server as pkiuser in Docker/Podman so that the log files will be owned by pkiuser as well.

The test has been updated to check the files created by the container and to verify that the container can be restarted successfully.

https://github.com/dogtagpki/pki/wiki/Deploying-PKI-Server-Container

The PKI server container (which is the base for all PKI
subsystem containers) has been updated to install the ROOT
webapp (which provides Web UI files) and PKI webapp (which
provides common PKI services), and store the default config
files.

The startup script has been modified to install the default
config files if it doesn't exist already, use PKI NSS CLI
instead of certutil to prepare the certs, and run the server
as pkiuser in Docker/Podman so that the log files will be
owned by pkiuser as well.

The test has been updated to check the files created by the
container and to verify that the container can be restarted
successfully.
@edewata edewata requested a review from fmarco76 May 23, 2024 18:44
Copy link

sonarcloud bot commented May 23, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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 May 28, 2024

@fmarco76 Thanks!

@edewata edewata merged commit dd5cc93 into dogtagpki:master May 28, 2024
143 of 148 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