Ori Livneh <ori.liv...@gmail.com> added the comment:

I updated the patch, making the emendations suggested by Eric's review, and 
making a few additional changes. I was bothered that translate_path was 
duplicating functionality from _url_collapse_path_split, so I made some 
corrections to the latter and rewrote translate_path so that it uses 
_url_collapse_path_split and implements only the checks necessary for 
translating the path to a path on the local file system path.

translate_path relies on os.path to make sure the path is safe, which is (I 
think) also the proper way to do things, since it's not good to have to 
remember to update path semantics in http.server if Python is ported to 
additional platforms.

I also read the various specs and made sure entities are unquoted at the 
appropriate point.

----------
Added file: http://bugs.python.org/file22338/translate_path_rev2.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11197>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to