revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 08:52:39 +01:00
Code Issues Releases Wiki Activity

Update nginx configuration documentation.

Browse source 
Summary: The configuration guide for nginx recommends using a conditional statement to check if a file exists, which is listed on the [[http://wiki.nginx.org/Pitfalls#Check_IF_File_Exists | nginx wiki]] as a common pitfall. It is not necessary to use a conditional and probably should not be recommended to do so.

Test Plan: We use this nginx configuration in our installation.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D9401
This commit is contained in:
Joshua Spence 2014-06-07 12:08:07 -07:00 committed by epriestley
parent 858d1e699f
commit 3219039575
1 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/docs/user/configuration/configuration_guide.diviner
View file

@ -85,18 +85,15 @@ For nginx, use a configuration like this:
name=nginx.conf
server {
server_name phabricator.example.com;
root /path/to/phabricator/webroot;
try_files $uri $uri/ /index.php;
root /path/to/phabricator/webroot;
location / {
index index.php;
index index.php;
rewrite ^/(.*)$ /index.php?__path__=/$1 last;
}
if ( !-f $request_filename )
{
rewrite ^/(.*)$ /index.php?__path__=/$1 last;
break;
}
location = /favicon.ico {
try_files $uri =204;
}
location /index.php {