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

Add chown to default nginx conf file, to make it owned by nginx user #576

Merged

Conversation

mattmatician
Copy link
Contributor

@mattmatician mattmatician commented Aug 15, 2023

Change owner of /etc/nginx/conf.d/default.conf to nginx:root

This change enables /etc/nginx/conf.d/default.conf changes through docker-entrypoint scripts.

/docker-entrypoint.d scripts are unable to write to /etc/nginx/conf.d/default.conf unless the owner of the file is changed to nginx.

The owner of /etc/nginx/conf.d/default.confis nginx:root in the nginxinc/nginx-unprivileged image.

Addressed Issue

Fixes #575

Checklist

Copy link
Member

@nscuro nscuro left a comment

Choose a reason for hiding this comment

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

Thanks @mattmatician!

@nscuro nscuro added this to the 4.9 milestone Oct 11, 2023
@nscuro nscuro merged commit 365ab9f into DependencyTrack:master Oct 11, 2023
8 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.

Unable to edit Nginx config with entrypoint script
2 participants