Nick Coghlan <ncogh...@gmail.com> added the comment:

Initially, because I was suggesting the names shadow the subprocess convenience 
functions so they *had* to live in a different namespace.

However, even after changing the names to explicitly include "shell", I'd like 
to keep them away from the general subprocess functionality - these wrappers 
are more convenient for shell operations than the subprocess ones, but it's 
that very convenience that makes them potentially dangerous in larger 
applications that may be interpolating data that untrusted users can manipulate.

Since the intended audience is system administrators working on shell-like 
operations, the shell utility module seems like an appropriate place for them. 
Both the "import shutil" and the "shell" in the names would then serve as red 
flags on a code review or security audit.

----------

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

Reply via email to