Skip to content

Small modifications to make the nginx conf more universal #1

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Small modifications to make the nginx conf more universal #1

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 31, 2015

I have changed the configuration for nginx to read all php files and not only app_dev.php
I have also changed the way it redirects, because now nginx sends everything to fpm.

Thanks for your comments !

@jverdeyen
Copy link
Member

Thanks @emmanuellgd for your interest in our Nginx docker image. It's better to fork our repository and start an automated build for your version. Or create a custom build image while inheriting from our image.

You can find more about automated builds on the docker docs, https://docs.docker.com/docker-hub/builds/.

I'll test your modifications and check if we can apply them on our image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant