Mercurial > p > roundup > code
diff roundup/cgi/apache.py @ 4908:92757447dcf0 routing
Remove PATH_INFO hacks that strip leading slash
PATH_INFO should always start with '/' when not empty, see
http://tools.ietf.org/html/rfc3875#section-4.1.5
Instead, do leading slash stripping inside cgi.client
| author | anatoly techtonik <techtonik@gmail.com> |
|---|---|
| date | Wed, 16 Jul 2014 03:10:02 +0300 |
| parents | 302bc481ceb1 |
| children |
line wrap: on
line diff
--- a/roundup/cgi/apache.py Tue Jul 15 13:47:28 2014 +0300 +++ b/roundup/cgi/apache.py Wed Jul 16 03:10:02 2014 +0300 @@ -121,11 +121,6 @@ # Note: cookies are read from HTTP variables, so we need all HTTP vars req.add_common_vars() _env = dict(req.subprocess_env) - # XXX classname must be the first item in PATH_INFO. roundup.cgi does: - # path = string.split(os.environ.get('PATH_INFO', '/'), '/') - # os.environ['PATH_INFO'] = string.join(path[2:], '/') - # we just remove the first character ('/') - _env["PATH_INFO"] = req.path_info[1:] if _timing: _env["CGI_SHOW_TIMING"] = _timing _form = cgi.FieldStorage(req, environ=_env)