geoffrey.al...@gmail.com wrote:
This is the second version of the patch, including the suggestion from Daniel.
Thank you! Looks good to me. I did a basic 'smoke test', just as far as checking that during its first few 'svn log' commands it does indeed hide the password in its output.
Committed in http://svn.apache.org/r1863262 . - Julian
Log message: [[[ Do not print password to screen in svn_load_dirs.pl. * contrib/client-side/svn_load_dirs/svn_load_dirs.pl.in (sanitize_pwd): New function. (safe_read_from_pipe, read_from_process): Update the sites printing the command line to screen to use sanitize_pwd. (usage): Warn that other local OS users may be able to see the password passed on the command-line. Fix indentation; that is, replace the 2 tab occurrences by 8 spaces. ]]]