Commit dbb369d0 authored by Senthil Kumaran's avatar Senthil Kumaran

3.2- fix the incorrect changes made for PATH_INFO value - Issue10484

parent 935b6296
...@@ -850,14 +850,7 @@ def _url_collapse_path_split(path): ...@@ -850,14 +850,7 @@ def _url_collapse_path_split(path):
# Filter out blank non trailing parts before consuming the '..'. # Filter out blank non trailing parts before consuming the '..'.
path_parts = [part for part in path_parts[:-1] if part] + path_parts[-1:] path_parts = [part for part in path_parts[:-1] if part] + path_parts[-1:]
if path_parts: if path_parts:
# Special case for CGI's for PATH_INFO tail_part = path_parts.pop()
if path.startswith('/cgi-bin') or path.startswith('/htbin'):
tail_part = []
while path_parts[-1] not in ('cgi-bin','htbin'):
tail_part.insert(0,path_parts.pop())
tail_part = "/".join(tail_part)
else:
tail_part = path_parts.pop()
else: else:
tail_part = '' tail_part = ''
head_parts = [] head_parts = []
...@@ -952,8 +945,11 @@ class CGIHTTPRequestHandler(SimpleHTTPRequestHandler): ...@@ -952,8 +945,11 @@ class CGIHTTPRequestHandler(SimpleHTTPRequestHandler):
""" """
splitpath = _url_collapse_path_split(self.path) splitpath = _url_collapse_path_split(self.path)
if splitpath[0] in self.cgi_directories: joined_path = '/'.join(splitpath)
self.cgi_info = splitpath dir_sep = joined_path.find('/',1)
head, tail = joined_path[:dir_sep], joined_path[dir_sep+1:]
if head in self.cgi_directories:
self.cgi_info = head, tail
return True return True
return False return False
......
...@@ -377,7 +377,6 @@ class CGIHTTPServerTestCase(BaseTestCase): ...@@ -377,7 +377,6 @@ class CGIHTTPServerTestCase(BaseTestCase):
'/.//': ('/', ''), '/.//': ('/', ''),
'cgi-bin/file1.py': ('/cgi-bin', 'file1.py'), 'cgi-bin/file1.py': ('/cgi-bin', 'file1.py'),
'/cgi-bin/file1.py': ('/cgi-bin', 'file1.py'), '/cgi-bin/file1.py': ('/cgi-bin', 'file1.py'),
'/cgi-bin/file1.py/PATH-INFO': ('/cgi-bin', 'file1.py/PATH-INFO'),
'a': ('/', 'a'), 'a': ('/', 'a'),
'/a': ('/', 'a'), '/a': ('/', 'a'),
'//a': ('/', 'a'), '//a': ('/', 'a'),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment