This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author quentel
Recipients orsenthil, quentel, v+python
Date 2012-05-07.06:14:12
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1336371253.84.0.553175626218.issue14565@psf.upfronthosting.co.za>
In-reply-to
Content
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()"
History
Date User Action Args
2012-05-07 06:14:13quentelsetrecipients: + quentel, orsenthil, v+python
2012-05-07 06:14:13quentelsetmessageid: <1336371253.84.0.553175626218.issue14565@psf.upfronthosting.co.za>
2012-05-07 06:14:13quentellinkissue14565 messages
2012-05-07 06:14:13quentelcreate