path_info equals script_name on lighttpd using fastcgi under a subdirectory when accessed without a trailing '/'

[mtrichardson]

I have a Django app running under lighttpd and fastcgi in a subdirectory. When accessed without a trailing '/', eg, example.com/carfactory instead of example.com/carfactory/, script_name is the same as path_info resulting in an attempt to resolve 'carfactory/carfactory' instead of '/'.

Simple patch attached that fixes this.

[mtrichardson]

Simple patch which tests to see if path_info equals script_name and, if so, set it to '/'

[mtrichardson]

This is only relevant for flup 1.0.1 - flup 1.0 appears to handle this properly.

[mtredinnick]

Fixed in [8569].

[qingfeng]

​http://code.djangoproject.com/ticket/8874

nginx + django Still have this problem

[kmtracey]

Replying to qingfeng:

​http://code.djangoproject.com/ticket/8874

nginx + django Still have this problem

Why did you reopen this ticket if that other one (also open) already reports the issue? We only need one open ticket per problem. This one was fixed nearly a year ago. After that long, it is preferable to open a new ticket to track a new problem. Unless there is already an open ticket that covers the issue (#8874?), in which case it is not necessary to do anything unless you want to pitch in and help with fixing the open ticket.

[qingfeng]

This patch is not good.

the code is good:

if not path_info:
    path_info = u'/'
if path_info == script_name:
    script_name = u''

[qingfeng]

nginx.conf

[qingfeng]

Django project conf file

[qingfeng]

nginx+django project conf file:

​http://code.djangoproject.com/attachment/ticket/8490/nginx.conf

​http://code.djangoproject.com/attachment/ticket/8490/sina.conf

urls.py:

from django.conf.urls.defaults import *

urlpatterns = patterns('',
    (r'^hello/', 'demo.views.hello'),
)

demo/views.py

from django.http import HttpResponse

def hello(request):
    return HttpResponse("Hello")

Django 1.1 Test:

curl -I ​http://localhost/hello

Restuls:

HTTP/1.1 404 NOT FOUND
Server: nginx/0.7.62
Date: Wed, 16 Sep 2009 09:23:29 GMT
Content-Type: text/html
Connection: keep-alive

use new patch:

​http://code.djangoproject.com/attachment/ticket/8490/wsgi_r11339.diff

curl -I ​http://localhost/hello

Results:

HTTP/1.1 200 OK
Server: nginx/0.7.62
Date: Wed, 16 Sep 2009 10:27:11 GMT
Content-Type: text/html; charset=utf-8
Connection: keep-alive
Vary: Cookie

[kmtracey]

Please stop reopening this ticket to report problems with nginx. This ticket reported a problem that was fixed before 1.0. That fix may have caused a different problem, reported in #8874. #8874 is the correct place to track fixing the follow-on problem, not here.

[qingfeng]

Replying to kmtracey:

Please stop reopening this ticket to report problems with nginx. This ticket reported a problem that was fixed before 1.0. That fix may have caused a different problem, reported in #8874. #8874 is the correct place to track fixing the follow-on problem, not here.

sorry,i see.

[jacob]

Milestone 1.0 deleted

[Aymeric Augustin <aymeric.augustin@…>]

In f5dbb566ee07bf3328015c70574b832052b4ebb7:

Fixed #17550 -- Removed a workaround for a bug in flup 1.0.1.

This reverts commit 948a833eb7fd9be0bc2d585c92a407f036ef42b2.

flup appears to be dead, and this fix breaks legitimate uses of Django.

Refs #8490.