lib: Refresh nginx/uwsgi configuration

Thomas Monjalon · stephenfin · commit dc7bf1cc3c8c · 2016-09-07T21:03:37.000+01:00
As stated in http://uwsgi-docs.readthedocs.io/en/latest/Nginx.html: " Unfortunately nginx is not able to rewrite PATH_INFO accordingly to SCRIPT_NAME. For such reason you need to instruct uWSGI to map specific apps in the so called “mountpoint” and rewrite SCRIPT_NAME and PATH_INFO automatically [...] ancient uWSGI versions used to support the so called “uwsgi_modifier1 30” approach. Do not do it. it is a really ugly hack " Replacing the uwsgi_modifier1 hack by a mount point seems to work. Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
diff --git a/lib/nginx/patchwork.conf b/lib/nginx/patchwork.conf
@@ -35,7 +35,6 @@ http {
         location / {
             include uwsgi_params;
             uwsgi_pass unix:/run/uwsgi/patchwork.sock;
-            uwsgi_modifier1 30;
         }
     }
 }
diff --git a/lib/uwsgi/patchwork.ini b/lib/uwsgi/patchwork.ini
@@ -6,11 +6,16 @@ project = patchwork
 base = /opt
 user = www-data
 group = www-data
+url = /
 
 chdir = %(base)/%(project)
 pythonpath = %(base)/%(project)
 module = %(project).wsgi:application
 
+# rewrite PATH_INFO and SCRIPT_NAME according to mount-points (for nginx)
+manage-script-name = true
+mount = %(url)=%(module)
+
 master = true
 processes = 5
 # increase buffer size to avoid "502 bad gateway error"

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,6 @@ http {`
`35`	`35`	`location / {`
`36`	`36`	`include uwsgi_params;`
`37`	`37`	`uwsgi_pass unix:/run/uwsgi/patchwork.sock;`
`38`		`- uwsgi_modifier1 30;`
`39`	`38`	`}`
`40`	`39`	`}`
`41`	`40`	`}`