[issue10484] http.server.is_cgi fails to handle CGI URLs containing PATH_INFO

Glenn Linderman report at bugs.python.org
Fri Mar 16 09:31:29 CET 2012


Glenn Linderman <v+python at g.nevcal.com> added the comment:

Senthil, the patch you submitted breaks the encapsulation of cgi_directories -- this should be changeable per particular application via subclassing or assignment, but you've built those directories into the code.  Perhaps you should take a look at my code for is_cgi (which I was trying to post while you were posting the patches, sorry I was slow).

Perhaps there should be one more test, which actually changes the cgi_directories, and tests a few more things, to prove that the code does not hard code those directories (it shouldn't).

----------
status: closed -> open

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue10484>
_______________________________________


More information about the Python-bugs-list mailing list