We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Right now you can browse the /wp-content/uploads directory of a site deployed from this image.
/wp-content/uploads
I've fixed this by adding the following line to each of my derived images:
RUN sed -i 's/Indexes //g' /etc/apache2/apache2.conf
I'm sure there's a less dangerous way of doing the above, but I'm thinking that disabling indexes should be the default behavior. Thoughts?
The text was updated successfully, but these errors were encountered:
You are right, Indexes should be turned off. Maybe just a change in the .htaccess would be simpler?
.htaccess
Sorry, something went wrong.
Sounds good to me, especially since we're already using .htaccess for apache configuration.
Shouldn't we just change the default config in php:apache directly?
daghack
Successfully merging a pull request may close this issue.
Right now you can browse the
/wp-content/uploads
directory of a site deployed from this image.I've fixed this by adding the following line to each of my derived images:
RUN sed -i 's/Indexes //g' /etc/apache2/apache2.conf
I'm sure there's a less dangerous way of doing the above, but I'm thinking that disabling indexes should be the default behavior. Thoughts?
The text was updated successfully, but these errors were encountered: