[issue38863] Improve is_cgi() in http.server

2019-12-04 Thread Siwon Kang
Siwon Kang added the comment: Obviously, I should have taken back what I added to cgi_directories. Thank you Pablo and Victor! -- ___ Python tracker <https://bugs.python.org/issue38

[issue38863] Improve is_cgi() in http.server

2019-11-20 Thread Siwon Kang
Siwon Kang added the comment: Hi Donghee, Since you said this is not a bug, I changed the title describing this is a matter of improvement. For your comment, I would say sorry first that I have made you confused. My mention about apache is just to give you an example for the other module

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- pull_requests: -16785 ___ Python tracker <https://bugs.python.org/issue38863> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- pull_requests: +16800 pull_request: https://github.com/python/cpython/pull/17312 ___ Python tracker <https://bugs.python.org/issue38

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Siwon Kang added the comment: Thank you for your message and the info about 21323. I agree with the idea that cgi files are conventionally placed at the cgi-bin of the root but there is no explicit regulation so other servers, apache for example, handle this kind of sub directories

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
Change by Siwon Kang : -- keywords: +patch pull_requests: +16785 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17292 ___ Python tracker <https://bugs.python.org/issu

[issue38863] http.server is_cgi() does not correctly separate dir

2019-11-20 Thread Siwon Kang
New submission from Siwon Kang : is_cgi() in CGIHTTPRequestHandler class separates given path into (dir, rest) then checks if dir is in cgi_directories. However, it divides based on the first seen '/', multi-level directories like /sub/dir/cgi-bin/hello.py is divided into head=/sub, rest=dir