Message160119
Thanks for the explanation
I still think that the patch can be simplified, not using path lengths and the "found" flag
collapsed_path = _url_collapse_path(self.path)
for head in self.cgi_directories:
if head==collapsed_path:
self.cgi_info = (head,'')
return True
elif collapsed_path.startswith(head) \
and collapsed_path[len(head)]=='/':
self.cgi_info = head, collapsed_path[len(head)+1:]
return True
return False
BTW the last "return False" is rather useless since is_cgi() is only used in tests like "if is_cgi()" |
|
Date |
User |
Action |
Args |
2012-05-07 06:14:13 | quentel | set | recipients:
+ quentel, orsenthil, v+python |
2012-05-07 06:14:13 | quentel | set | messageid: <1336371253.84.0.553175626218.issue14565@psf.upfronthosting.co.za> |
2012-05-07 06:14:13 | quentel | link | issue14565 messages |
2012-05-07 06:14:13 | quentel | create | |
|