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

Audit permissions set by Dockerfile #79

Open
jonoliver opened this issue Oct 5, 2023 · 0 comments
Open

Audit permissions set by Dockerfile #79

jonoliver opened this issue Oct 5, 2023 · 0 comments

Comments

@jonoliver
Copy link
Member

Revisit the need for the chown and chmod commands here:

# So we can update plugins
RUN chown -R www-data:www-data /var/www
RUN find /var/www/ -type d -exec chmod 0755 {} \;
RUN find /var/www/ -type f -exec chmod 644 {} \;

  • These steps can take quite a while when building the image, and the find commands appear to be overly broad.
  • The comment only states so we can update plugins, but it's unclear what the initial issue was.

Can these lines be removed or modified?

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

No branches or pull requests

1 participant