TrafeX/docker-php-nginx

Clean up nginx config

jduan00 opened this issue · 2 comments

jduan00 commented

Just want to share two small proposed changes:

  1. Remove /etc/nginx/fastcgi.conf as it is not used and likely cause confusion. fastcgi_params is actually used.
  2. Use the following line for relevant references (ex. /etc/nginx/conf.d/default.conf, fastcgi_params)
fastcgi_param   SCRIPT_FILENAME         $request_filename;
TrafeX commented

Hi @jduan00,

Thank you for the tips. I've looked into it and came to the following;

  1. It's indeed not used, but it's part of the default nginx installation. I prefer to leave this alone because this container image serves as an example to adjust to your needs and I want to keep it simple and as standard as possible.
  2. That's indeed a good alternative for $document_root$fastcgi_script_name, but it's doesn't do exactly the same. I'm afraid this can create a backwards compatibility break, while it isn't a major improvement.

What I did change is removing the duplicate SCRIPT_NAME: b829e43

  • 👍1
jduan00 commented

Hi @TrafeX,

Thanks for the following up. It is matter of personal taste. I want to make it a min set of files within the container, thus I took the axe to get rid of anything that is not used.

Also, I appreciate you pointed out the difference about SCRIPT_FILENAME. https://serverfault.com/questions/465607/nginx-document-rootfastcgi-script-name-vs-request-filename

However, for our Laravel based PHP apps, the one works is:
fastcgi_param SCRIPT_FILENAME $request_filename;

Cheers,

Jack

  • 👍1