Project

General

Profile

Bug #1489

mod_fastcgi "/"-prefix extension issue

Added by icy almost 9 years ago. Updated over 8 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
mod_fastcgi
Target version:
Start date:
Due date:
% Done:

0%

Missing in 1.5.x:

Description

A bug in mod_fastcgi.c prevents fastcgi settings with the /-prefix as extension not work properly in some situations.

Patch attached

mod_fastcgi.c.patch View (594 Bytes) icy, 2007-12-17 18:22

mod_scgi.c.patch View - mod_scgi patch for same flaw (578 Bytes) icy, 2008-01-16 00:00

mod_proxy.c.patch View - mod_proxy patch for same flaw (940 Bytes) icy, 2008-01-16 00:01

Associated revisions

Revision bdd2fce4 (diff)
Added by glen over 8 years ago

- fixed a bug that made /-prefixed fcgi extensions being handled also when matching the end of the uri (#1489)

git-svn-id: svn://svn.lighttpd.net/lighttpd/branches/lighttpd-1.4.x@2030 152afb58-edef-0310-8abb-c4023f1b3aa9

Revision 513057eb (diff)
Added by glen over 8 years ago

- fix also scgi and proxy modules. #1489

git-svn-id: svn://svn.lighttpd.net/lighttpd/branches/lighttpd-1.4.x@2035 152afb58-edef-0310-8abb-c4023f1b3aa9

History

#1 Updated by admin almost 9 years ago

What situations?

#2 Updated by icy over 8 years ago

suppose one has a fcgi setting like the following:

fastcgi.server = ( "/fcgi.py" => (....) )

One would expect (having read the docs) that URIs like "/fcgi.py" or "/fcgi.py/foo/bar" are handled by the fcgi backend. That is true, but URIs like "/foo/fcgi.py" will also be forwarded to the fcgi backend because any fcgi extension setting using the /-prefix will match the internal check if it is at the beginning OR THE END of the URI.

The "/foo/fcgi.py" behaviour is contrary to what the docs say because it should ONLY match if at the beginning of the URI.

#3 Updated by glen over 8 years ago

applied in r2030 for 1.4

#4 Updated by glen over 8 years ago

  • Status changed from New to Fixed
  • Resolution set to fixed

#5 Updated by glen over 8 years ago

  • Status changed from Fixed to Need Feedback
  • Resolution deleted (fixed)

#6 Updated by glen over 8 years ago

  • Status changed from Need Feedback to New

#7 Updated by glen over 8 years ago

Committed revision r2035.

#8 Updated by glen over 8 years ago

  • Status changed from New to Fixed
  • Resolution set to fixed

Also available in: Atom