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).